Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dataconnect: relax LocalDateSerializer encoding and decoding, and add unit test coverage #6451

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

dconeybe
Copy link
Contributor

@dconeybe dconeybe commented Nov 9, 2024

No description provided.

Copy link
Contributor

github-actions bot commented Nov 9, 2024

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

Copy link
Contributor

github-actions bot commented Nov 9, 2024

Vertex AI Mock Responses Check ⚠️

A newer major version of the mock responses for Vertex AI unit tests is available. update_responses.sh should be updated to clone the latest version of the responses: v5.2

Copy link
Contributor

github-actions bot commented Nov 9, 2024

Test Results

   58 files  + 4     58 suites  +4   2m 17s ⏱️ +19s
  538 tests +23    537 ✅ +23  1 💤 ±0  0 ❌ ±0 
1 076 runs  +46  1 074 ✅ +46  2 💤 ±0  0 ❌ ±0 

Results for commit aaad6fd. ± Comparison against base commit f20340a.

@dconeybe dconeybe merged commit 0eff5d3 into main Nov 12, 2024
41 checks passed
@dconeybe dconeybe deleted the dconeybe/dataconnect/LocalDateUnitTests branch November 12, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants