-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Bump version of main
to 12.0.0; execute announced deprecations
#10883
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
Conversation
One thing that's missing here and will come in a follow-up is removing support for ansible-core 2.16. (That will also involve moving ansible-core 2.17 into the EOL CI.) |
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.
One fix and a possible adjustment. Thanks for putting it all together.
- Note that V(auto) will be deprecated in a future version. | ||
# TODO: Deprecate 'auto' in community.general 13.0.0 |
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.
Why not
- Note that V(auto) will be deprecated in a future version. | |
# TODO: Deprecate 'auto' in community.general 13.0.0 | |
- Note that V(auto) will be deprecated in community.general 13.0.0. | |
# TODO: Deprecate 'auto' in community.general 13.0.0 |
?
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.
I'd keep this open until we decide (by merging an appropriate PR) to actually deprecate it. We might forget to merge a PR that goes into 13.0.0 :)
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.
okie, makes sense
Co-authored-by: Alexei Znamensky <[email protected]>
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.
LGTM
@russoz thanks for reviewing! |
SUMMARY
Remove announced features and apply announced breaking changes for 12.0.0.
ISSUE TYPE
COMPONENT NAME
various