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

Allow opting into specific feature flags #2931

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

vinistock
Copy link
Member

@vinistock vinistock commented Nov 28, 2024

Motivation

In the implementation of feature flags, I forgot to check if the user opted into a specific flag. Currently, the code doesn't check for that, which means you can only opt into all of them.

Implementation

We just need to check if the user opted into that specific flag.

Automated Tests

Added a test.

@vinistock vinistock added vscode This pull request should be included in the VS Code extension's release notes bugfix This PR will fix an existing bug labels Nov 28, 2024 — with Graphite App
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@vinistock vinistock requested review from alexcrocha, andyw8 and st0012 and removed request for andyw8, st0012 and alexcrocha November 28, 2024 15:56
@vinistock vinistock marked this pull request as ready for review November 28, 2024 15:57
@vinistock vinistock requested a review from a team as a code owner November 28, 2024 15:57
@vinistock vinistock merged commit dc805d9 into main Nov 28, 2024
37 checks passed
Copy link
Member Author

Merge activity

  • Nov 28, 11:22 AM EST: A user merged this pull request with Graphite.

@vinistock vinistock deleted the 11-28-allow_opting_into_specific_feature_flags branch November 28, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This PR will fix an existing bug vscode This pull request should be included in the VS Code extension's release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants