-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[Fixes #13087] Remove flower - this comes from old monitoring solution #13060
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
Conversation
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Fabian Fischer.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #13060 +/- ##
==========================================
+ Coverage 61.96% 67.97% +6.01%
==========================================
Files 980 980
Lines 59303 59303
Branches 6932 6932
==========================================
+ Hits 36745 40314 +3569
+ Misses 20976 17340 -3636
- Partials 1582 1649 +67 🚀 New features to boost your workflow:
|
@giohappy i agree with this, can we merge? |
@mattiagiupponi I agree too, I added the |
@Gpetrak please add an issue and fix the title accoridngly. |
FYI the base image has been rebuilt and published to Docker Hub |
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.