search

HEPCloud / decisionengine

HEPCloud Decision Engine framework
Apache License 2.0
6 stars 26 forks source link

Set upper limit version for flake8. #664

Closed vitodb closed 2 years ago

vitodb commented 2 years ago

This is needed to have pytest-flake8 and flake8 versions working together.

codecov[bot] commented 2 years ago

Codecov Report

Merging #664 (004cde6) into master (9879741) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #664   +/-   ##
=======================================
  Coverage   97.27%   97.27%           
=======================================
  Files          50       50           
  Lines        3120     3120           
  Branches      501      501           
=======================================
  Hits         3035     3035           
  Misses         54       54           
  Partials       31       31
Flag Coverage Δ
python-3.10 97.21% <ø> (+0.12%) :arrow_up:
python-3.6 96.97% <ø> (+0.06%) :arrow_up:

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

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

jcpunk commented 2 years ago

I'd be inclined to switch this to pre-commit via - https://flake8.pycqa.org/en/latest/user/using-hooks.html

so we can streamline things locally...

Might want to move pylint there too: https://pylint.pycqa.org/en/latest/user_guide/installation/pre-commit-integration.html