treeherder
treeherder copied to clipboard
mozilla
Show common alert ID
I have some suggestions:
- I would say that "(log in as a sheriff to create)" should not be displayed there since the common alert already exists.
- Alert number for common alerts is orange and for the usual alerts is blue colored.
* Alert number for common alerts is orange and for the usual alerts is blue colored.
isn't already orange?
* Alert number for common alerts is orange and for the usual alerts is blue colored.isn't already orange?
In my opinion you should keep the original color, which is blue. You already have the "Common alert" text to differentiate the alerts and I don't see a reason to not keep the original design.
Codecov Report
Base: 90.75% // Head: 90.75% // No change to project coverage :thumbsup:
Coverage data is based on head (
1a38aa7) compared to base (4ad7db9). Patch has no changes to coverable lines.
Additional details and impacted files
@@ Coverage Diff @@
## master #7541 +/- ##
=======================================
Coverage 90.75% 90.75%
=======================================
Files 294 294
Lines 14966 14966
=======================================
Hits 13582 13582
Misses 1384 1384
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
if you rebase this on upstream/master, you will fix the issue with circleci: builds.
please rebase on master branch to pick up changes to fix the issue with the broken builds job.