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

Add feature flag support to GlobalState #2826

Merged

Conversation

vinistock
Copy link
Member

@vinistock vinistock commented Nov 4, 2024

Motivation

This PR starts passing the enabled feature flags to the server and storing it in the global state. That way the Ruby LSP and its add-ons can check for specific feature flags. This also allows users of other editors to opt into features through initializationOptions.

Implementation

  • Started passing all enabled flag information to the server
  • Started storing this information in the global state

Automated Tests

Added unit tests to verify feature flag processing and retrieval functionality.

Copy link
Member Author

vinistock commented Nov 4, 2024

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

Join @vinistock and the rest of your teammates on Graphite Graphite

@vinistock vinistock changed the title Start passing enabled feature flags to server Add feature flag support to GlobalState Nov 4, 2024
@vinistock vinistock requested review from andyw8 and st0012 November 4, 2024 21:45
@vinistock vinistock added server This pull request should be included in the server gem's release notes enhancement New feature or request labels Nov 4, 2024 — with Graphite App
@vinistock vinistock marked this pull request as ready for review November 4, 2024 21:45
@vinistock vinistock requested a review from a team as a code owner November 4, 2024 21:45
@vinistock vinistock force-pushed the 11-04-add_the_ability_to_rollout_features_to_a_percetange_of_users branch from 718ef4d to 547631f Compare November 5, 2024 15:09
@vinistock vinistock force-pushed the 11-04-start_passing_enabled_feature_flags_to_server branch from c729b55 to 70540b6 Compare November 5, 2024 15:09
@vinistock vinistock changed the base branch from 11-04-add_the_ability_to_rollout_features_to_a_percetange_of_users to graphite-base/2826 November 5, 2024 18:18
lib/ruby_lsp/global_state.rb Outdated Show resolved Hide resolved
lib/ruby_lsp/global_state.rb Outdated Show resolved Hide resolved
@vinistock vinistock force-pushed the 11-04-start_passing_enabled_feature_flags_to_server branch from 70540b6 to b62cc31 Compare November 5, 2024 20:30
@vinistock vinistock changed the base branch from graphite-base/2826 to main November 5, 2024 20:31
@vinistock vinistock merged commit 6f83f74 into main Nov 5, 2024
57 checks passed
Copy link
Member Author

Merge activity

  • Nov 5, 3:56 PM EST: A user merged this pull request with Graphite.

@vinistock vinistock deleted the 11-04-start_passing_enabled_feature_flags_to_server branch November 5, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request server This pull request should be included in the server gem's release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants