Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch test requirement to pytest-flake8. #75

Merged
merged 3 commits into from
Feb 24, 2019
Merged

Switch test requirement to pytest-flake8. #75

merged 3 commits into from
Feb 24, 2019

Conversation

smarlowucf
Copy link
Contributor

Flake8 runs both pyflakes and pycodestyle. Pep8 is deprecated and renamed to pycodestyle.

@codecov
Copy link

codecov bot commented Aug 7, 2018

Codecov Report

Merging #75 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #75   +/-   ##
=======================================
  Coverage   98.64%   98.64%           
=======================================
  Files          53       53           
  Lines        1105     1105           
=======================================
  Hits         1090     1090           
  Misses         15       15

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 2487d0e...dccb93b. Read the comment docs.

Flake8 runs both pyflakes and codestyle. Pep8 was renamed to
and supported as codestyle.
@smarlowucf
Copy link
Contributor Author

Reference to name change: PyCQA/pycodestyle#466

@smarlowucf
Copy link
Contributor Author

Not sure why PyPy is having trouble...

@p1c2u
Copy link
Collaborator

p1c2u commented Aug 14, 2018

I downgraded pytest pytest-cov but it didn't help. Really strange.

@smarlowucf
Copy link
Contributor Author

I saw the change in the spec-validator also caused a slow down. The only shared differences are flake8 and pycodestyle. Pytest with coverage + xdist used to cause problems in PyPy due to deadlock but that's been fixed for a while and xdist isn't used here...

@p1c2u p1c2u merged commit 922fce4 into python-openapi:master Feb 24, 2019
bjmc pushed a commit to bjmc/openapi-core that referenced this pull request Jun 12, 2019
Switch test requirement to pytest-flake8.
Sign up for free to 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.

2 participants