-
Notifications
You must be signed in to change notification settings - Fork 59
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
Accept more prefixes for PR titles #463
Comments
Maybe we could have prefixes related to deprecating code as well. What is the recommended prefix for code deprecation PRs at the moment, |
I guess |
|
Addressed in #505. |
Currently, we have the following:
hi-ml/.github/workflows/check-pr-title.yml
Line 14 in c185873
We might want to be more precise or accurate sometimes. I've seen others that could be useful. For example:
TEST
: to add or modify testsFIX
: to fix something that is not necessarily aBUG
such as a typoMNT
: maintenance, to upgrade a package version (ENH: Upgrade PyTorch Lightning to 1.6.4 #453) or stuff related to packaging, linting, CI, etc.STY
: as a shorter alternative toSTYLE
REL
: for release commitsWIP
: for work in progress (I don't like this, I think using drafts PRs is better)COMP
: fix e.g. warnings (I think it comes from "compiling" or similar)PERF
: improve performanceSome sources:
The text was updated successfully, but these errors were encountered: