-
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
docs - adapted abadger's changelog announcement email into dev process #48172
Conversation
Signed-off-by: Adam Miller <admiller@redhat.com>
Hi @maxamillion, thank you for submitting this pull-request! |
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.
Thanks for doing this!
Changelogs | ||
---------- | ||
|
||
Please add a changelog fragment with any PR that is |
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.
Bullet points for needs/does not need list could make this more readable
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.
Nice addition. As we discussed on IRC, this PR should also deduplicate material here vs. material on https://docs.ansible.com/ansible/devel/reference_appendices/release_and_maintenance.html#changelogs, and link between them.
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.
Thanks for adding this to the Dev guide. I always have to search for it and never remember where it was last time.
Signed-off-by: Adam Miller <admiller@redhat.com>
Signed-off-by: Adam Miller <admiller@redhat.com>
Big "Thank You!" to @acozine for picking this up and running with it while I was on PTO. Does this need anything else or is it good to merge? |
I think it's ready to go, especially since @gundalow is going to be making additional changes to this page and this section. I think the changes in this PR set the ground-work for the next round of changes. |
@maxamillion Thank you for doing this, it's a big help |
ansible#48172) * docs - adapted abadger's changelog announcement email into dev process Signed-off-by: Adam Miller <admiller@redhat.com> * migrate changelog docs to one place, add ref to old location Signed-off-by: Adam Miller <admiller@redhat.com> * add changes based on feedback Signed-off-by: Adam Miller <admiller@redhat.com> * Refines changelog docs, minor re-org of page * moves links and anchors for new content * Further reorg of page * fixes header per jamescassell
ansible#48172) * docs - adapted abadger's changelog announcement email into dev process Signed-off-by: Adam Miller <admiller@redhat.com> * migrate changelog docs to one place, add ref to old location Signed-off-by: Adam Miller <admiller@redhat.com> * add changes based on feedback Signed-off-by: Adam Miller <admiller@redhat.com> * Refines changelog docs, minor re-org of page * moves links and anchors for new content * Further reorg of page * fixes header per jamescassell
Signed-off-by: Adam Miller admiller@redhat.com
SUMMARY
Add docs for developer guides about changelog fragments
ISSUE TYPE
COMPONENT NAME
docs
ANSIBLE VERSION