search

samdoran / ansible-collection-macos

Ansible collection of roles and modules for use on macOS
8 stars 2 forks source link

Remove deprecated warn parameter #27

Closed samdoran closed 1 year ago

samdoran commented 1 year ago

Fixes #26

codecov[bot] commented 1 year ago

Codecov Report

Merging #27 (1f89969) into main (39efffd) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   35.49%   35.49%           
=======================================
  Files           5        5           
  Lines         386      386           
  Branches       48       48           
=======================================
  Hits          137      137           
  Misses        249      249
Flag Coverage Δ
sanity 32.34% <ø> (ø)
units 82.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

webknjaz commented 1 year ago

Try restarting the tests? Maybe, Galaxy is drunk...

samdoran commented 1 year ago

No, this is due to Galaxy NG and old versions of Ansible Core. CI has been broken for a while. There was a fix in 2.12, 2.13, and 2.14. I'm not sure how to make collection installation work for older versions of Ansible Core, though.

samdoran commented 1 year ago

Removing the collection dependencies is an acceptable workaround for now.