-
Notifications
You must be signed in to change notification settings - Fork 177
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
Overhaul the complete labelling system #4438
Conversation
133fca2
to
b2e4868
Compare
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.
These changes make sense to me, thanks for centralizing this all in one place! It'll be nice to have the labels apply equally across all PRs now 😄
a6a7834
to
ccb315b
Compare
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! Nice to solve so many issues at once 🚀
Fixes
Fixes #3823 by @dhruvkb
Fixes #4203 by @AetherUnbound
Fixes WordPress/openverse-infrastructure#740 by @dhruvkb
Description
This PR overhauls the complete labelling system to unify it in one place instead of the various disconnected systems.
Now the end-to-end labelling process is handled by the PR automations workflow.
Credits
Thanks to @GauranshMathur for bringing this up.
Checklist
Update index.md
).main
) or a parent feature branch.just catalog/generate-docs
for catalogPRs) or the media properties generator (
just catalog/generate-docs media-props
for the catalog or
just api/generate-docs
for the API) where applicable.Developer Certificate of Origin
Developer Certificate of Origin