-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
chore: Replace hex colors in constants file with tokens #8330
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
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! Ideally this PR should have screenshots of the changes so we can be 100% confident there are no visual regressions
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8330 +/- ##
==========================================
+ Coverage 39.83% 40.33% +0.50%
==========================================
Files 1233 1236 +3
Lines 29821 29976 +155
Branches 2840 2881 +41
==========================================
+ Hits 11880 12092 +212
+ Misses 17252 17188 -64
- Partials 689 696 +7 ☔ View full report in Codecov by Sentry. |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Description
Related issues
Fixes: #8331
Manual testing steps
Screenshots/Recordings
N/A
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist