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 `disable` state to generated policy documentation #52

Closed duckpuppy closed 4 years ago

duckpuppy commented 4 years ago

Description

This PR adds support for documenting disabled rules to the generated policy documentation.

Testing Done

New unit tests, local testing using the example repos included in this project. A new disabled rule was added to both example sets.

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 4 years ago

Codecov Report

Merging #52 into master will increase coverage by 0.12%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage   57.61%   57.73%   +0.12%     
==========================================
  Files           8        8              
  Lines        1385     1389       +4     
  Branches      261      262       +1     
==========================================
+ Hits          798      802       +4     
  Misses        586      586              
  Partials        1        1              
Impacted Files Coverage Δ
manheim_c7n_tools/policygen.py 99.75% <100.00%> (+<0.01%) :arrow_up:
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 7cec202...df006bf. Read the comment docs.