-
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
♻️Clusters: make non dev feature, disabled by default (⚠️) #4129
♻️Clusters: make non dev feature, disabled by default (⚠️) #4129
Conversation
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.
thanks!
Codecov Report
@@ Coverage Diff @@
## master #4129 +/- ##
=========================================
- Coverage 84.6% 78.0% -6.6%
=========================================
Files 953 398 -555
Lines 41364 20612 -20752
Branches 957 174 -783
=========================================
- Hits 35018 16096 -18922
+ Misses 6128 4467 -1661
+ Partials 218 49 -169
Flags with carried forward coverage won't be shown. Click here to find out more.
|
af84968
to
2e9440d
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.
Bravo!
and sorry for the inconvenience of app_environments
4801868
to
eaf827f
Compare
eaf827f
to
bf5a4f2
Compare
Code Climate has analyzed commit bf5a4f2 and detected 0 issues on this pull request. View more on Code Climate. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
What do these changes do?
Since the WEBSERVER_DEV_ENABLED env seems to bring some issues, this PR brings
WEBSERVER_CLUTERS
out of it.By default it is disabled, and must be explicitly enabled.
Related issue/s
How to test
DevOps Checklist