Skip to content

feat(v2): add IsFeatureEnabled#454

Merged
quartzmo merged 3 commits intogoogleapis:mainfrom
quartzmo:feature-gating
Dec 16, 2025
Merged

feat(v2): add IsFeatureEnabled#454
quartzmo merged 3 commits intogoogleapis:mainfrom
quartzmo:feature-gating

Conversation

@quartzmo
Copy link
Copy Markdown
Member

Check if an experimental feature is enabled via environment variable.

The feature name must be the suffix, e.g., FOO for GOOGLE_SDK_GO_EXPERIMENTAL_FOO.

Check if an experimental feature is enabled via environment variable.
The feature name must be the suffix, e.g., FOO for
GOOGLE_SDK_GO_EXPERIMENTAL_FOO.
@quartzmo quartzmo requested a review from a team December 16, 2025 18:26
Comment thread v2/feature.go
Copy link
Copy Markdown
Contributor

@westarle westarle left a comment

Choose a reason for hiding this comment

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

I'm a novice in Go so feel free to push back (e.g. if tests for experimental features only run in separate environments) -- we should state how tests interact with this in the comment.

Comment thread v2/feature.go
Comment thread v2/feature.go
westarle
westarle previously approved these changes Dec 16, 2025
Copy link
Copy Markdown
Contributor

@westarle westarle left a comment

Choose a reason for hiding this comment

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

Optional notes, otherwise great!

Comment thread v2/feature.go
Comment thread v2/feature.go
@quartzmo quartzmo merged commit 2700b8a into googleapis:main Dec 16, 2025
4 checks passed
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.

4 participants