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

feat(gengapic): generate helpers for Go 1.23 iterators #1542

Merged
merged 4 commits into from
Aug 16, 2024

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Jun 28, 2024

This changes starts generating build tag protected adapters for using our iterator types in Go range expressions.

Related: googleapis/gax-go#358

This changes starts generating build tag protected adapters for
using our iterator types in Go range expressions.

Related: googleapis/gax-go#358
@codyoss codyoss requested review from a team as code owners June 28, 2024 20:25
@codyoss codyoss added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 28, 2024
@codyoss
Copy link
Member Author

codyoss commented Jun 28, 2024

requires gax changes to be fully released and rolled out to clients

@codyoss codyoss removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 24, 2024
@codyoss
Copy link
Member Author

codyoss commented Jul 24, 2024

The new gax libraries have been rolled out to all clients. This is now safe for review.

Copy link
Collaborator

@noahdietz noahdietz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. Let's just make sure we don't merge this until we are ready to have it in our clients e.g. in case we need to fix something and cut a release and this would be in that release.

@codyoss
Copy link
Member Author

codyoss commented Jul 24, 2024

Agreed. Let's wait until 1.23 is out, which should be in about 2-3 weeks.

@codyoss codyoss added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 24, 2024
@codyoss codyoss removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 15, 2024
@codyoss codyoss requested a review from quartzmo August 15, 2024 21:33
@codyoss codyoss merged commit d7ed683 into googleapis:main Aug 16, 2024
6 checks passed
@codyoss codyoss deleted the go123-iter branch August 16, 2024 18:34
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.

3 participants