debrief / pepys-import

Support library for Pepys maritime data analysis environment
https://pepys-import.readthedocs.io/
Apache License 2.0
5 stars 5 forks source link

Fix black style issues, and pin black version #1127

Closed robintw closed 2 years ago

robintw commented 2 years ago

🧰 Issue

N/A

🚀 Overview:

This PR fixes some black style issues that were causing the CI tests to fail, and pins the black version to the latest stable version (22.1.0) in both the pre-commit config file, and the requirements_dev.txt file.

🤔 Reason:

Get the same black version everywhere we use it (locally and CI)

🔨Work carried out:

Confirmations

📝 Developer Notes:

If you're going to be contributing code that you've edited on your local machine @IanMayo, it's probably worth doing a pip install -U black to upgrade. pre-commit should upgrade its version automatically next time it's used.

codecov[bot] commented 2 years ago

Codecov Report

Merging #1127 (1596b6f) into develop (3f625e3) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1127   +/-   ##
========================================
  Coverage    80.10%   80.10%           
========================================
  Files          110      110           
  Lines        11931    11931           
========================================
  Hits          9557     9557           
  Misses        2374     2374           
Impacted Files Coverage Δ
importers/eag_importer.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 85b7587...1596b6f. Read the comment docs.