-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
update changelog fragment guidance with examples #77040
Conversation
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.
This looks great! I've added some suggestions to make it even better.
Co-authored-by: Alicia Cozine <879121+acozine@users.noreply.github.com>
.. code-block:: yaml | ||
|
||
breaking_changes: | ||
- ec2_instance - instance wait for state behavior has changed. If plays require the old behavior of waiting for the instance monitoring status to become OK when launching a new instance, the action will need to specify ``state: started`` (https://github.com/ansible-collections/amazon.aws/pull/481). |
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.
This snippet seems inconsistent with the paragraph above. The explanation says to use the present tense but the example has a mixture and starts with the past tense.
Should the guidance be made clearer? Does the "present tense" request imply an imperative mood?
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.
sorry about that, updated in #77138
SUMMARY
Community wanted better guidance on how to write good changelog fragments. This PR
Implements ansible-community/community-topics#64 where this was decided on.
There are two guides now - one for core and one for collections, thus the repeated content in this PR.
ISSUE TYPE
COMPONENT NAME
docs/docsite/rst/community/collection_development_process.rst
docs/docsite/rst/community/development_process.rst
ADDITIONAL INFORMATION