POT icon indicating copy to clipboard operation
POT copied to clipboard

[WIP] pre-commit with ruff,codespell,yamlint

Open rflamary opened this issue 1 year ago • 1 comments

Types of changes

This very large PR introduce a new style for the coe thta follow ruff formating with a fats pre-commit that will hopfully avoid "pep8" cmmts in the PRs in the future

Currently implements:

  • [x] Ruff Lint (for main code and tetss/exmaples and doc)
  • [x] Codespell (found many typos)
  • [x] yamllint

Motivation and context / Related issue

How has this been tested (if it applies)

PR checklist

  • [ ] I have read the CONTRIBUTING document.
  • [ ] The documentation is up-to-date with the changes I made (check build artifacts).
  • [ ] All tests passed, and additional code has been covered with new tests.
  • [ ] I have added the PR and Issue fix to the RELEASES.md file.

rflamary avatar Jul 18 '24 13:07 rflamary

Codecov Report

Attention: Patch coverage is 94.44444% with 46 lines in your changes missing coverage. Please review.

Project coverage is 96.67%. Comparing base (d0849a4) to head (284c500). Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #660      +/-   ##
==========================================
- Coverage   96.67%   96.67%   -0.01%     
==========================================
  Files          85       85              
  Lines       16956    16948       -8     
==========================================
- Hits        16392    16384       -8     
  Misses        564      564              

codecov[bot] avatar Jul 18 '24 14:07 codecov[bot]