Skip to content
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

[PR #1971/efb2c671 backport][stable-2.18] Add rules on moving content to collections outside of Ansible #2053

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Oct 24, 2024

This is a backport of PR #1971 as merged into devel (efb2c67).

This is basically the proposal of https://forum.ansible.com/t/5738/2.

I was not able to find a rule in the requirements that if content is moved out from collections in Ansible, that the resulting collection also has to be part of Ansible. Since the original discussions are more than two years old (ansible-community/community-topics#167), I'm missing the context... If someone remembers (or finds where this is mentioned in the requirements or elsewhere in the docs), please mention it here :)

* Add rules on moving content to collections outside of Ansible.

* Apply suggestions from code review.

Co-authored-by: Maxwell G <[email protected]>

* One more from gotmax23.

* Reference section on CLAs.

---------

Co-authored-by: Maxwell G <[email protected]>
(cherry picked from commit efb2c67)
@patchback patchback bot requested a review from a team as a code owner October 24, 2024 18:32
@ansible-documentation-bot ansible-documentation-bot bot added the sc_approval This PR requires approval from the Ansible Community Steering Committee label Oct 24, 2024
@samccann samccann merged commit bcfdeab into stable-2.18 Oct 24, 2024
11 checks passed
@samccann samccann deleted the patchback/backports/stable-2.18/efb2c671ffb0b083025d5cb384555c0f1f261678/pr-1971 branch October 24, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sc_approval This PR requires approval from the Ansible Community Steering Committee
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants