Add reminder that pin must be in sync with two other places - Githubissues

cisagov / skeleton-ansible-role

A skeleton project for quickly getting a new cisagov Ansible role started.

Creative Commons Zero v1.0 Universal

7 stars 6 forks source link

Add reminder that pin must be in sync with two other places #191

Closed jsf9k closed 6 months ago

jsf9k commented 6 months ago

🗣 Description

This pull request modifies a comment to remind us that the ansible-core pin must stay in sync across this repo as well as:

💭 Motivation and context

If we update the pin in one place we must remember to update it in the two other places.

🧪 Testing

All automated tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

jsf9k commented 6 months ago

We can close this pull request because a new version of the ansible-core Python package has been released that does not suffer from the bug discussed in ansible/ansible#82702. This bug broke any symlinked files in vars, tasks, etc. for any Ansible role installed via ansible-galaxy.

See also #193.