-
Notifications
You must be signed in to change notification settings - Fork 1
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
Edit the README but not the CHANGELOG #36
base: master
Are you sure you want to change the base?
Conversation
After using the PR template for a while, I think simplifying it will be helpful: - GitHub includes the commit message description automatically with the aim of saving you time as it is sometimes also a good PR description. With the current template, I need to cut this test and paste it into the description section (which is really annoying!). - Make a single simpler checklist. Add information as comment and have a straightforward list which helps us remembering the changelog, tests and documentation without needing to invest much time. The changelog bulletpoint will also be used in GitHub Actions.
Request changes in a PR without CHANGELOG update.
`Ana06/automatic-pull-request-review@v0.1.0` is a fork of https://github.com/AndrewMusgrave/automatic-pull-request-review which fixes `DISMISS` and provides an `allow_duplicate` option which allows to only approve once.
Ana06/get-changed-files@v1.1 is a fork of https://github.com/jitterbit/get-changed-files, which supports `pull_request_target` and allow to filter files using regular expressions. As we need to use `pull_request_target`, Ana06/get-changed-files@v1.1 works, but jitterbit/get-changed-files@v1 doesn't.
Doing evil. 😈
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 add bug fixes, new features, breaking changes and anything else you think it is worthwhile mentioning to the master (unreleased)
section of CHANGELOG.md. If no CHANGELOG update is needed add the following to the PR description: [x] No CHANGELOG update needed
CHANGELOG updated or no update needed, thanks! 😄
bef6797
to
16d39e6
Compare
Doing evil. 😈
Checklist