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

feat(ci): add ansible 7, 8 to test matrix #151

Merged
merged 4 commits into from
Sep 25, 2023
Merged

Conversation

JonasPammer
Copy link
Owner

as per https://docs.ansible.com/ansible/latest/reference_appendices/release_and_maintenance.html#ansible-core-support-matrix, minimum controller python version still is 3.9 at least for the documented 2.15

i couldn't see anything breaking in interest for me in molecule.
i hope my changed tox deps declaration works as i expect it to (i.e. "if ansible-4, use moleucle == 4., otherwhise use >= 5.")
also changed to >= to be always up to date. bit rot checked by CI anyways

Closes #115

as per https://docs.ansible.com/ansible/latest/reference_appendices/release_and_maintenance.html#ansible-core-support-matrix, minimum controller python version still is 3.9 at least for the documented 2.15

i couldn't see anything breaking in interest for me in molecule
i hope my changed deps works as i expect it to (i.e. "if ansible-4, use moleucle == 4.*, otherwhise use >= 5.*" -- didnt want to make it that verbose by adding numbers there too.
also changed to >= to be always up to date. bit rot checked by CI
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 25, 2023
@JonasPammer JonasPammer changed the title feat(ci): add ansible 7, 8, 9 to test matrix feat(ci): add ansible 7, 8 to test matrix Sep 25, 2023
i dont like tox's docs much
@JonasPammer JonasPammer marked this pull request as ready for review September 25, 2023 13:28
@JonasPammer JonasPammer merged commit dfa90c2 into master Sep 25, 2023
2 checks passed
@JonasPammer JonasPammer deleted the JonasPammer/issue111 branch September 25, 2023 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: add ansible 7 to test matrix
1 participant