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

Pinned pycodestyle #25701

Merged
merged 2 commits into from
Mar 13, 2019
Merged

Pinned pycodestyle #25701

merged 2 commits into from
Mar 13, 2019

Conversation

WillAyd
Copy link
Member

@WillAyd WillAyd commented Mar 13, 2019

Fixes issues with the LINT failures as of recent. Quick fix with more permanent solutions split between #25697 and #25622

@TomAugspurger

@WillAyd WillAyd added the Build Library building on various platforms label Mar 13, 2019
@TomAugspurger
Copy link
Contributor

TomAugspurger commented Mar 13, 2019

With the latest flake8 (3.7.7) and pycodestyle=2.4.0, I get an exception

module 'pycodestyle' has no attribute 'maximum_doc_length

I'm having trouble finding which versions of flake8 are compatible with which versions of pycodestyle.

Hopefully conda is smart enough to solve these properly.

@TomAugspurger
Copy link
Contributor

Ok, for reference here's what conda found

flake8                    3.6.0                    py37_0
pycodestyle               2.4.0                    py37_0
pyflakes                  2.0.0                    py37_0

and that works well.

@codecov
Copy link

codecov bot commented Mar 13, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #25701   +/-   ##
=======================================
  Coverage   91.28%   91.28%           
=======================================
  Files         173      173           
  Lines       52967    52967           
=======================================
  Hits        48350    48350           
  Misses       4617     4617
Flag Coverage Δ
#multiple 89.86% <ø> (ø) ⬆️
#single 41.74% <ø> (ø) ⬆️

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 479f821...449a0e7. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Mar 13, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #25701   +/-   ##
=======================================
  Coverage   91.28%   91.28%           
=======================================
  Files         173      173           
  Lines       52967    52967           
=======================================
  Hits        48350    48350           
  Misses       4617     4617
Flag Coverage Δ
#multiple 89.86% <ø> (ø) ⬆️
#single 41.74% <ø> (ø) ⬆️

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 479f821...449a0e7. Read the comment docs.

@WillAyd
Copy link
Member Author

WillAyd commented Mar 13, 2019

@TomAugspurger all green here. Would we need to back port this as well?

@jreback jreback modified the milestones: 0.25.0, 0.24.2 Mar 13, 2019
@jreback jreback merged commit c57d162 into pandas-dev:master Mar 13, 2019
@jreback
Copy link
Contributor

jreback commented Mar 13, 2019

thanks @WillAyd

yeah this needs a backport too

@lumberbot-app
Copy link

lumberbot-app bot commented Mar 13, 2019

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

  1. Checkout backport branch and update it.
$ git checkout 0.24.x
$ git pull
  1. Cherry pick the first parent branch of the this PR on top of the older branch:
$ git cherry-pick -m1 c57d162139c629a5e90bec6c1f336606755a0f7e
  1. You will likely have some merge/cherry-pick conflict here, fix them and commit:
$ git commit -am 'Backport PR #25701: Pinned pycodestyle'
  1. Push to a named branch :
git push YOURFORK 0.24.x:auto-backport-of-pr-25701-on-0.24.x
  1. Create a PR against branch 0.24.x, I would have named this PR:

"Backport PR #25701 on branch 0.24.x"

And apply the correct labels and milestones.

Congratulation you did some good work ! Hopefully your backport PR will be tested by the continuous integration and merged soon!

If these instruction are inaccurate, feel free to suggest an improvement.

@jreback
Copy link
Contributor

jreback commented Mar 13, 2019

we may need to patch https://github.com/macpython/pandas-wheels as well

@WillAyd WillAyd deleted the pin-pycodestyle branch March 13, 2019 16:16
WillAyd added a commit to WillAyd/pandas that referenced this pull request Mar 13, 2019
@WillAyd WillAyd mentioned this pull request Mar 13, 2019
sighingnow added a commit to sighingnow/pandas that referenced this pull request Mar 14, 2019
* master: (22 commits)
  Fixturize tests/frame/test_operators.py (pandas-dev#25641)
  Update ValueError message in corr (pandas-dev#25729)
  DOC: fix some grammar and inconsistency issues in the User Guide (pandas-dev#25728)
  ENH: Add public start, stop, and step attributes to RangeIndex (pandas-dev#25720)
  Make Rolling.apply documentation clearer (pandas-dev#25712)
  pandas-dev#25707 - Fixed flakiness in stata write test (pandas-dev#25714)
  Json normalize nan support (pandas-dev#25619)
  TST: resolve issues with test_constructor_dtype_datetime64 (pandas-dev#24868)
  DEPR: Deprecate box kwarg for to_timedelta and to_datetime (pandas-dev#24486)
  BUG: Preserve name in DatetimeIndex.snap (pandas-dev#25585)
  Fix concat not respecting order of OrderedDict (pandas-dev#25224)
  CLN: remove pandas.core.categorical (pandas-dev#25655)
  TST/CLN: Remove more Panel tests (pandas-dev#25675)
  Pinned pycodestyle (pandas-dev#25701)
  DOC: update date of 0.24.2 release notes (pandas-dev#25699)
  BUG: Fix error in replace with strings that are large numbers (pandas-dev#25616) (pandas-dev#25644)
  BUG: fix usage of na_sentinel with sort=True in factorize() (pandas-dev#25592)
  BUG: Fix to_string output when using header (pandas-dev#16718) (pandas-dev#25602)
  CLN: Remove unused test code (pandas-dev#25670)
  CLN: remove Panel from concat error message (pandas-dev#25676)
  ...

# Conflicts:
#	doc/source/whatsnew/v0.25.0.rst
@WillAyd WillAyd mentioned this pull request Mar 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants