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

update CI matrix for modules with new ansible upstream stable branch #808

Merged
merged 2 commits into from
Sep 26, 2022

Conversation

D3DeFi
Copy link
Contributor

@D3DeFi D3DeFi commented Sep 23, 2022

SUMMARY

Fixes #811

Add stable-2.14 tests back as mentioned in the ansible-collections/news-for-maintainers#24

I also went ahead and removed 2.10 (EOL) and 2.11 (EOL Nov/22). Now, each version in the matrix has exclude for python2.7/3.6. I wonder if I should remove those python versions as well.

Edit: it seems that python2.7 and python3.6 are indeed no longer tested after this patch so I am removing them as well from the testing matrix.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

CI

@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Merging #808 (9b52a63) into main (953c494) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #808      +/-   ##
==========================================
- Coverage   77.63%   77.42%   -0.21%     
==========================================
  Files          29       29              
  Lines        3859     3859              
  Branches     1043     1043              
==========================================
- Hits         2996     2988       -8     
- Misses        543      548       +5     
- Partials      320      323       +3     
Impacted Files Coverage Δ
...ity/zabbix/plugins/modules/zabbix_template_info.py 80.51% <0.00%> (-2.60%) ⬇️
.../community/zabbix/plugins/module_utils/_version.py 38.18% <0.00%> (-1.82%) ⬇️
...ommunity/zabbix/plugins/modules/zabbix_template.py 78.06% <0.00%> (-1.49%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Andersson007
Copy link
Contributor

Fixes #811

@D3DeFi D3DeFi merged commit 8fa08c2 into ansible-collections:main Sep 26, 2022
@D3DeFi D3DeFi deleted the ci-matrix branch September 26, 2022 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collection CI related actions needed
2 participants