-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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: Revamp of labels #10317
base: main
Are you sure you want to change the base?
chore: Revamp of labels #10317
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
030d60e
to
0c8c1fc
Compare
36f5863
to
f515954
Compare
f515954
to
feabc98
Compare
- name: dependencies | ||
color: 276bd1 | ||
description: Something is not working as expected. | ||
color: '000000' |
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.
Is this color change intentional?
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.
I feel black would stand out more as a bug. Welcome to feedback on this.
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.
I don't think i've actually seen black in the ui, so....what's it look like 🤷
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.
Absolutely not
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10317 +/- ##
===========================================
- Coverage 50.37% 40.28% -10.09%
===========================================
Files 228 184 -44
Lines 20641 16898 -3743
Branches 1240 953 -287
===========================================
- Hits 10397 6807 -3590
+ Misses 10199 10046 -153
Partials 45 45
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
not sure on the black for bug but otherwise lgtm
Important
The label synchronisation workflow must be temporarily disabled for this to merge.
Some labels were removed. Examples:
If
interactions
was renamed totype:interactions
, wouldn't that remove all the labels from previous issues and pull requests?Yes. However, that can be avoided with the following steps:
crazy-max/ghaction-github-labeler will then identify that
type:interactions
exists and will not delete any labels, but make amendments, if any.How will the deduplication of "api changes" and "api support" work?
Similarily to the above question, we can rename "api changes" to "type:discord update" and those with that label will be updated. However, we cannot rename "api support" to "type:discord update" as label names must be unique.
We can automatically modify all those issues and pull requests. For example, with issues:
Or, we could do nothing. Both are effortless.
I'll be doing the manual work once this merges. No pressure on anyone else!
This is a time to make any other label changes! Open to suggestions.