Skip to content

Commit 6bf855e

Browse files
committed
Update merge request message, per @wesleybl feedback
1 parent c05f79d commit 6bf855e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

docs/contributing/index.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,8 @@ See the [tool documentation](https://github.com/plone/meta) for more information
168168
If you ignore a check, then write the good reason either in a comment or in the merge message.
169169
- Get at least one approval from another Plone contributor, preferably a member of the GitHub Team for the repository.
170170
- If the pull request has documentation, ask for a review from `@plone/documentation-team`.
171-
- You don't need to write anything in the merge message, as it usually populates with all the commit messages.
171+
- Usually the merge message populates with all the commit messages, but some repositories aren't configured to do this.
172+
It's helpful to include a descriptive message in the merge request.
172173
- After merging, look for any linked issues that should be closed, and close them.
173174
Sometimes the contributor forgets to include `closes #123` in the description, which would automatically close the related issue.
174175
- If the change is a bugfix, make sure that it is not merged to an older release branch unless it is also merged in separate pull request to `main`.

0 commit comments

Comments
 (0)