-
Notifications
You must be signed in to change notification settings - Fork 27
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
Create Plugin: Allow setting feature flags via cli args #984
Conversation
f879305
to
0f5704c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one, I think we needed this for a long time! 👏
Left two comments, but generally LGTM 👍
Co-authored-by: Levente Balogh <[email protected]>
51417fd
to
9ef654f
Compare
changed approach making review stale
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
8ce0a9a
to
6a79d7c
Compare
🚀 PR was released in |
What this PR does / why we need it:
This PR allows users to set feature flags via cli args when scaffolding or updating a plugin. This makes it much easier for users when scaffolding as they don't need to scaffold, create or change the
cprc json
file, then run@grafana/create-plugin update
to update the configs. It also makes it easier for contributors to test or add new features behind flags.Feature flags can be passed using the
feature-flags
arg. Multiple can be comma separated. Any unknown flags will produce a warning:Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Use the below canary release feature flags can be set using an arg like
--feature-flags="bundleGrafanaUI"
.📦 Published PR as canary version:
Canary Versions
✨ Test out this PR locally via: