-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Update labels and URLs for outdated-icons #10802
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.
Additionally to the following comments, the field labels
of the template .github/ISSUE_TEMPLATE/icon_update.yml must be updated.
@@ -8,11 +8,11 @@ body: | |||
attributes: | |||
value: | | |||
Before opening a new issue, make sure it isn't covered by an existing issue. | |||
Please search for [issues with the `icon outdated` label][icon-outdated-issues] | |||
Please search for [issues with the `update-icon/data` label][update-icon/data] |
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.
Please search for [issues with the `update-icon/data` label][update-icon/data] | |
Please search for [issues with the `update icon/data` label][update-icon/data] |
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.
Yes, let's drop the PR in favor of @adamrusted's #10804.
I still have concerns about those label naming changes. I feel that our labels are not as accurate as before. We can no longer find all issues related to icon updates or icon removal through a certain label. The semantics are not as easy to understand as before. These may not be friendly to our first-time contributors and issue authors. If you want to adopt these changes, you can review this one instead: |
The label in URLs should be updated since we renamed it.