Remove unnecessary flake8 exclude#416

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

gsnedders merged 1 commit intohtml5lib:masterfromjdufresne:flake8

Apr 16, 2019

Merged

Remove unnecessary flake8 exclude#416

gsnedders merged 1 commit intohtml5lib:masterfromjdufresne:flake8

Apr 16, 2019

Conversation

Copy link

Contributor

jdufresne commentedApr 6, 2019

Modern flake8 has a good default for exclude that includes the tox
directory.

https://gitlab.com/pycqa/flake8/blob/2b333fad1abe0bdb2e04132eabb0822e6ce63888/src/flake8/defaults.py#L4-14

Remove unnecessary flake8 exclude

f8e0601

Modern flake8 has a good default for exclude that includes the toxdirectory.https://gitlab.com/pycqa/flake8/blob/2b333fad1abe0bdb2e04132eabb0822e6ce63888/src/flake8/defaults.py#L4-14

Copy link

codecov-io commentedApr 6, 2019•
edited
Loading

Codecov Report

Merging#416 intomaster willnot change coverage.
The diff coverage isn/a.

@@           Coverage Diff           @@##           master     #416   +/-   ##=======================================  Coverage   90.79%   90.79%           =======================================  Files          50       50             Lines        6952     6952             Branches     1328     1328           =======================================  Hits         6312     6312             Misses        481      481             Partials      159      159

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update4f92357...f8e0601. Read thecomment docs.