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

fix(internal/postprocessor): add ability to override release level #8643

Merged
merged 1 commit into from Oct 4, 2023

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Oct 4, 2023

For stable client without services there is not a good way to infer the release level. Eventually it may be nice if this info is pushed into the service config rather than a blaze rule so we can extract it better. For now, lets a a manual override.

In the past for things like oslogin we just did not provide a service config entry. Although this would work here it is not ideal as there is still valuable metadata and configuration options in these files.

For stable client without services there is not a good way to infer
the release level. Eventually it may be nice if this info is pushed
into the service config rather than a blaze rule so we can extract
it better. For now, lets a a manual override.

In the past for things like oslogin we just did not provide a service
config entry. Although this would work here it is not ideal as there
is still valuable metadata and configuration options in these files.
@codyoss codyoss requested a review from a team as a code owner October 4, 2023 15:35
@product-auto-label product-auto-label bot added the size: s Pull request size is small. label Oct 4, 2023
Copy link
Member

@quartzmo quartzmo left a comment

Choose a reason for hiding this comment

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

Sweet! Thank you!

@codyoss codyoss merged commit 26c608a into googleapis:main Oct 4, 2023
9 checks passed
@codyoss codyoss deleted the release-lvl-or branch October 4, 2023 16:35
gcf-merge-on-green bot pushed a commit that referenced this pull request Oct 19, 2023
🤖 I have created a release *beep* *boop*
---


## [0.110.9](https://togithub.com/googleapis/google-cloud-go/compare/v0.110.8...v0.110.9) (2023-10-19)


### Bug Fixes

* **/Users/chrisdsmith/oss/google-cloud-go:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **all:** Update golang.org/x/net to v0.17.0 ([#8705](https://togithub.com/googleapis/google-cloud-go/issues/8705)) ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/aliasgen:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/examples/fake:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/gapicgen:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/generated/snippets:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/godocfx:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/postprocessor:** Add ability to override release level ([#8643](https://togithub.com/googleapis/google-cloud-go/issues/8643)) ([26c608a](https://togithub.com/googleapis/google-cloud-go/commit/26c608a8204d740767dfebf6aa473cdf1873e5f0))
* **internal/postprocessor:** Add missing assignment ([#8646](https://togithub.com/googleapis/google-cloud-go/issues/8646)) ([d8c5746](https://togithub.com/googleapis/google-cloud-go/commit/d8c5746e6dde1bd34c01a9886804f861c88c0cb7))
* **internal/postprocessor:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
bhshkh pushed a commit that referenced this pull request Nov 3, 2023
🤖 I have created a release *beep* *boop*
---


## [0.110.9](https://togithub.com/googleapis/google-cloud-go/compare/v0.110.8...v0.110.9) (2023-10-19)


### Bug Fixes

* **/Users/chrisdsmith/oss/google-cloud-go:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **all:** Update golang.org/x/net to v0.17.0 ([#8705](https://togithub.com/googleapis/google-cloud-go/issues/8705)) ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/aliasgen:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/examples/fake:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/gapicgen:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/generated/snippets:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/godocfx:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))
* **internal/postprocessor:** Add ability to override release level ([#8643](https://togithub.com/googleapis/google-cloud-go/issues/8643)) ([26c608a](https://togithub.com/googleapis/google-cloud-go/commit/26c608a8204d740767dfebf6aa473cdf1873e5f0))
* **internal/postprocessor:** Add missing assignment ([#8646](https://togithub.com/googleapis/google-cloud-go/issues/8646)) ([d8c5746](https://togithub.com/googleapis/google-cloud-go/commit/d8c5746e6dde1bd34c01a9886804f861c88c0cb7))
* **internal/postprocessor:** Update golang.org/x/net to v0.17.0 ([174da47](https://togithub.com/googleapis/google-cloud-go/commit/174da47254fefb12921bbfc65b7829a453af6f5d))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants