From c4dc911d2686bfbec47fd5884484a4a48873b8d1 Mon Sep 17 00:00:00 2001 From: "patchback[bot]" <45432694+patchback[bot]@users.noreply.github.com> Date: Sun, 8 Aug 2021 11:48:01 +0200 Subject: [PATCH] Inform contributors on changelog fragments in CONTRIBUTING.md (#3167) (#3173) * Inform contributors on changelog fragments. * Mention docs-only PRs as well. (cherry picked from commit 87ba15fa4589fab466231f68d5e0122ca8d312d3) Co-authored-by: Felix Fontein --- CONTRIBUTING.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ba30ed1e02..a40dbd59eb 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -26,6 +26,7 @@ Also, consider taking up a valuable, reviewed, but abandoned pull request which * All commits of a pull request branch will be squashed into one commit at last. That does not mean you must have only one commit on your pull request, though! * Please try not to force-push if it is not needed, so reviewers and other users looking at your pull request later can see the pull request commit history. * Do not add merge commits to your PR. The bot will complain and you will have to rebase ([instructions for rebasing](https://docs.ansible.com/ansible/latest/dev_guide/developing_rebasing.html)) to remove them before your PR can be merged. To avoid that git automatically does merges during pulls, you can configure it to do rebases instead by running `git config pull.rebase true` inside the respository checkout. +* Make sure your PR includes a [changelog fragment](https://docs.ansible.com/ansible/devel/community/development_process.html#changelogs-how-to). (You must not include a fragment for new modules or new plugins, except for test and filter plugins. Also you shouldn't include one for docs-only changes. If you're not sure, simply don't include one, we'll tell you whether one is needed or not :) ) You can also read [our Quick-start development guide](https://github.com/ansible/community-docs/blob/main/create_pr_quick_start_guide.rst).