CMakePP / CMinx

Generates API documentation for CMake functions and macros
https://cmakepp.github.io/CMinx/
Apache License 2.0
14 stars 5 forks source link

Pep8 format #89

Closed AutonomicPerfectionist closed 2 years ago

AutonomicPerfectionist commented 2 years ago

Is this pull request associated with an issue(s)? Fixes #88

Description Reformats all Python code according to PEP8

codecov[bot] commented 2 years ago

Codecov Report

Merging #89 (7fb8979) into master (ba6d1aa) will decrease coverage by 0.01%. The diff coverage is 96.92%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   95.40%   95.39%   -0.02%     
==========================================
  Files           5        5              
  Lines         675      673       -2     
==========================================
- Hits          644      642       -2     
  Misses         31       31              
Flag Coverage Δ
unittests 95.39% <96.92%> (-0.02%) :arrow_down:

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

Impacted Files Coverage Δ
cminx/parser/aggregator.py 92.79% <85.71%> (-0.10%) :arrow_down:
cminx/__init__.py 90.47% <93.84%> (ø)
cminx/rstwriter.py 98.22% <98.16%> (+<0.01%) :arrow_up:
cminx/documenter.py 98.16% <100.00%> (ø)
cminx/parser/__init__.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 ba6d1aa...7fb8979. Read the comment docs.

AutonomicPerfectionist commented 2 years ago

Disclaimer: I didn't read this; I'm trusting it's all just formatting

Sorry, one of the automated changes broke a test so I'm fixing that quick. Also, since some of the lines were broken up into multiple so Codecov thinks coverage has gone down slightly so I may have to write an additional test to get that back up

AutonomicPerfectionist commented 2 years ago

@ryanmrichard unfortunately I've ran out of time for now, everything looks good to me for review except that codecov is still complaining. I can write another test later if desired though

ryanmrichard commented 2 years ago

I'm fine with the slight drop in code coverage.

github-actions[bot] commented 2 years ago

🚀 [bumpr] Bumped! New version:v0.0.2 Changes:v0.0.1...v0.0.2