Closed sseraj closed 2 years ago
Verified that the fprettify check fails without formatting and outputs which files need to be formatted
Merging #44 (93c3fd8) into main (85281e8) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #44 +/- ##
=======================================
Coverage 60.51% 60.51%
=======================================
Files 5 5
Lines 1616 1616
=======================================
Hits 978 978
Misses 638 638
:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more
Why not stick with the two space tab?
Previously, we had an initial 2 space tab and then 3 space tabs after that. We decided to use a 4 space tab to be consistent with our Python formatting.
Verified that a single file failure also fails the job
Purpose
Formatting the Fortran code with fprettify
Expected time until merged
1 week
Type of change
Testing
Current test pass
Checklist
flake8
andblack
to make sure the code adheres to PEP-8 and is consistently formatted