search

BlueBrain / NeuroTS

Topological Neuron Synthesis
https://neurots.readthedocs.io/en/stable
Apache License 2.0
36 stars 5 forks source link

CI: Move black, codespell, isort, pycodestyle and pydocstyle from tox to pre-commit #29

Closed adrien-berchet closed 2 years ago

adrien-berchet commented 2 years ago

This commit also formats the files according to the new rules and add commitlint in CI.

codecov[bot] commented 2 years ago

Codecov Report

Merging #29 (4283317) into main (26e0a98) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   97.20%   97.20%           
=======================================
  Files          34       34           
  Lines        1859     1859           
  Branches      295      295           
=======================================
  Hits         1807     1807           
  Misses         35       35           
  Partials       17       17
Flag Coverage Δ
pytest 97.20% <100.00%> (ø)

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

Impacted Files Coverage Δ
neurots/morphmath/utils.py 100.00% <ø> (ø)
neurots/extract_input/__init__.py 100.00% <100.00%> (ø)
adrien-berchet commented 2 years ago

Hehe ;-P Remember that you have to pip install it locally and call pre-commit install for optimal use. If you don't it will just behave as usual: the code is only checked in the CI.

adrien-berchet commented 2 years ago

Ah and if you think the checks are too strict (I added a few new ones 😇), we can discuss it :-)