Skip to content

CLN: Remove "flake8: noqa" from files #15842

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

Closed
wants to merge 3 commits into from

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Mar 30, 2017

Just some minor house-cleaning to cut down on the number of search results found here.

@gfyoung gfyoung changed the title CLN: Remove flake: noqa from files CLN: Remove "flake8: noqa" from files Mar 30, 2017
@codecov
Copy link

codecov bot commented Mar 30, 2017

Codecov Report

Merging #15842 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #15842      +/-   ##
==========================================
- Coverage   90.97%   90.96%   -0.02%     
==========================================
  Files         143      143              
  Lines       49442    49442              
==========================================
- Hits        44982    44973       -9     
- Misses       4460     4469       +9
Flag Coverage Δ
#multiple 88.72% <ø> (ø) ⬆️
#single 40.69% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
pandas/util/clipboard/exceptions.py 71.42% <ø> (ø) ⬆️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.56% <0%> (-0.1%) ⬇️

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 de589c2...5d1edeb. Read the comment docs.

@jreback jreback added this to the 0.20.0 milestone Mar 30, 2017
@jreback jreback closed this in 046d3be Mar 30, 2017
@jreback jreback added the Code Style Code style, linting, code_checks label Mar 30, 2017
@jreback
Copy link
Contributor

jreback commented Mar 30, 2017

thanks!

@gfyoung gfyoung deleted the flake8-noqa-clean branch March 31, 2017 08:31
jreback pushed a commit that referenced this pull request Apr 2, 2017
Another round of house-cleaning that builds off #15842.    xref <a
href="https://github.com/pandas-
dev/pandas/issues/12066#issuecomment-172285473">#12066  (comment)</a>
: the issue remains unresolved, but it does not seem entirely
necessary to disable style-checking on the entire file for that IMO.

Author: gfyoung <[email protected]>

Closes #15867 from gfyoung/flake8-noqa-clean and squashes the following commits:

0c84926 [gfyoung] CLN: Make tseries/common.py flake8-able
7a799ff [gfyoung] CLN: Make _version.py flake8-able
7087b64 [gfyoung] CLN: Make test_categorical.py flake8-able
5d5abf8 [gfyoung] CLN: Make test_categorical.py flake8-able
6ace90b [gfyoung] CLN: Make test_eval.py flake8-able
mattip pushed a commit to mattip/pandas that referenced this pull request Apr 3, 2017
Just some minor house-cleaning to cut down on the number of search
results found <a href="https://github.com/pandas-
dev/pandas/search?utf8=%E2%9C%93&q=flake8%3A+noqa&type=">here</a>.

Author: gfyoung <[email protected]>

Closes pandas-dev#15842 from gfyoung/flake8-noqa-clean and squashes the following commits:

5d1edeb [gfyoung] CLN: Make test_compat.py flake8-able
f9079ff [gfyoung] CLN: Make exceptions.py flake8-able
0e236f5 [gfyoung] CLN: Make test_format.py flake8-able
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Style Code style, linting, code_checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants