Closed curious-attempt-bunny closed 2 years ago
@mehaase I addressed the feedback.
I also added documentation for running the tests, added the flake checks to the makefile, and duplicated the flake8 configuration so that it's picked up out of the box by the MS Visual Studio plugin.
Merging #43 (a7db577) into main (958189a) will increase coverage by
0.58%
. The diff coverage is100.00%
.:exclamation: Current head a7db577 differs from pull request most recent head 2aae729. Consider uploading reports for the commit 2aae729 to get more accurate results
@@ Coverage Diff @@
## main #43 +/- ##
==========================================
+ Coverage 96.81% 97.40% +0.58%
==========================================
Files 5 5
Lines 220 270 +50
==========================================
+ Hits 213 263 +50
Misses 7 7
Impacted Files | Coverage Δ | |
---|---|---|
src/attack_flow/graphviz.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 958189a...2aae729. Read the comment docs.
@mehasse Cleaned up. Thanks for your time reviewing these changes.
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
19 Code Smells
No Coverage information
0.0% Duplication
Thank you for the contribution @curious-attempt-bunny! This will definitely be valuable to the community at large.
Fixes #42
What Changed
Before
After
Limitations