Skip to content

Suggestions for global feature flags#6198

Merged
timvisee merged 1 commit intoglobal-feature-flagsfrom
feature-flag-suggestions
Mar 19, 2025
Merged

Suggestions for global feature flags#6198
timvisee merged 1 commit intoglobal-feature-flagsfrom
feature-flag-suggestions

Conversation

@JojiiOfficial
Copy link
Copy Markdown
Contributor

@JojiiOfficial JojiiOfficial commented Mar 19, 2025

Two minor suggestions for global feature flags in #6194:

  • Private static Flags to have better control of initialization
  • Public function for accessing global flags with fallback to default values to prevent unwrapping everywhere

This is just a suggestion and I'm happy to discuss or discard if not needed!

@JojiiOfficial JojiiOfficial requested a review from timvisee March 19, 2025 08:40
@JojiiOfficial JojiiOfficial mentioned this pull request Mar 19, 2025
6 tasks
Copy link
Copy Markdown
Member

@timvisee timvisee left a comment

Choose a reason for hiding this comment

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

Good suggestion. Thank you!

@timvisee timvisee merged commit f30b79e into global-feature-flags Mar 19, 2025
@timvisee timvisee deleted the feature-flag-suggestions branch March 19, 2025 09:14
timvisee added a commit that referenced this pull request Mar 19, 2025
* Add global feature flags structure, load on start

* Feature flag use of new shard key format

* Enable feature flag for new shard key format by default in dev builds

* Prevent unwrapping everywhere; Single place for initialization (#6198)

* Disable shard key feature flag by default for now, it causes problems

---------

Co-authored-by: Jojii <[email protected]>
timvisee added a commit that referenced this pull request Mar 21, 2025
* Add global feature flags structure, load on start

* Feature flag use of new shard key format

* Enable feature flag for new shard key format by default in dev builds

* Prevent unwrapping everywhere; Single place for initialization (#6198)

* Disable shard key feature flag by default for now, it causes problems

---------

Co-authored-by: Jojii <[email protected]>
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.

2 participants