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

chore: bump google.golang.org/api dep #11257

Merged
merged 5 commits into from
Dec 10, 2024

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Dec 10, 2024

This notably removes ref to OC.

This notably removes ref to OC.
@codyoss codyoss requested review from jba, eliben, a team and shollyman as code owners December 10, 2024 19:02
@codyoss codyoss requested a review from gkevinzheng December 10, 2024 19:02
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Dec 10, 2024
codyoss added a commit to codyoss/google-cloud-go that referenced this pull request Dec 10, 2024
I removed the obvious open census tests that were failing in googleapis#11257.
Some others it was not clear to me if these functions should be
instrumented with OT or not, so I skipped them for now and will let
the spanner team decide what to do with those tests.

Related: googleapis#11230
gcf-merge-on-green bot pushed a commit that referenced this pull request Dec 10, 2024
I removed the obvious open census tests that were failing in #11257. Some others it was not clear to me if these functions should be instrumented with OT or not, so I skipped them for now and will let the spanner team decide what to do with those tests.

Related: #11230
@gcf-merge-on-green gcf-merge-on-green bot merged commit b278d5c into googleapis:main Dec 10, 2024
350 of 352 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 10, 2024
@codyoss codyoss deleted the bump1210-apiary branch December 10, 2024 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants