Skip to content

Add Black linting check to CI and add to test reqs - #170

Open
alexgleith wants to merge 3 commits into
masterfrom
add-black-linting
Open

Add Black linting check to CI and add to test reqs#170
alexgleith wants to merge 3 commits into
masterfrom
add-black-linting

Conversation

@alexgleith

@alexgleithalexgleith commented Jun 29, 2022

Copy link
Copy Markdown
Contributor

Not sure if we want to do this, but it's nice to have consistent formatting 😄

Note that this doesn't run for Python 3.5, because you can't install Black for that version of Python.

@codecov

codecovBot commented Jun 29, 2022

Copy link
Copy Markdown

Codecov Report

Merging #170 (7e5fea4) into master (4d9655d) will not change coverage.
The diff coverage is 84.29%.

@@ Coverage Diff @@## master #170 +/- ##
=======================================
Coverage 84.96% 84.96% =======================================
Files 12 12 Lines 1137 1137 Branches 156 156 =======================================
Hits 966 966 Misses 152 152 Partials 19 19 
Impacted FilesCoverage Δ
aodntools/ncwriter/__init__.py100.00% <ø> (ø)
...odntools/timeseries_products/gridded_timeseries.py0.00% <0.00%> (ø)
setup.py0.00% <0.00%> (ø)
aodntools/timeseries_products/common.py89.47% <81.25%> (ø)
...eseries_products/velocity_aggregated_timeseries.py91.85% <92.53%> (ø)
...tools/timeseries_products/aggregated_timeseries.py95.18% <94.44%> (ø)
.../timeseries_products/velocity_hourly_timeseries.py95.20% <95.18%> (ø)
aodntools/timeseries_products/hourly_timeseries.py98.19% <97.67%> (ø)
aodntools/__init__.py100.00% <100.00%> (ø)
aodntools/ncwriter/imos_template.py100.00% <100.00%> (ø)
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@alexgleith