manheim / manheim-c7n-tools

Manheim's Cloud Custodian (c7n) wrapper package, policy generator, runner, and supporting tools.
https://manheim-c7n-tools.readthedocs.io/
Apache License 2.0
45 stars 25 forks source link

Add check_deprecations='yes' for Validate step #69

Closed jleopold28 closed 2 years ago

jleopold28 commented 2 years ago

Description

Resolve Issue https://github.com/manheim/manheim-c7n-tools/issues/67

Testing Done

Tested c7n run with manheim/manheim-c7n-tools:pr-69 docker container. Confirmed that validate stage does not throw errors.

Important Notes

Optional; instructions to reviewers, plans for how to release this, dependencies in other repos, etc.

Contributor License Agreement

Required for external contributors.

By submitting this work for inclusion in manheim-c7n-tools, I agree to the following terms:

codecov-commenter commented 2 years ago

Codecov Report

Merging #69 (0655376) into master (7673b01) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 0655376 differs from pull request most recent head d95770d. Consider uploading reports for the commit d95770d to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #69   +/-   ##
=======================================
  Coverage   61.49%   61.49%           
=======================================
  Files           9        9           
  Lines        1470     1470           
  Branches      306      306           
=======================================
  Hits          904      904           
  Misses        565      565           
  Partials        1        1           
Impacted Files Coverage Δ
manheim_c7n_tools/runner.py 100.00% <ø> (ø)
manheim_c7n_tools/version.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3b9be68...d95770d. Read the comment docs.