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.

Sign up for GitHub

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

Jump to bottom

Pycodestyle is what has replaced pep8 #67

Merged
merged 2 commits into from
Nov 14, 2017
Merged

Pycodestyle is what has replaced pep8 #67

merged 2 commits into from
Nov 14, 2017

Conversation

Bachmann1234
Copy link
Owner

@Bachmann1234 Bachmann1234 commented Nov 14, 2017 •
edited
Loading

Future versions of diff-cover wont support pep8 as it is being replaced with pycodestyle which seems to be basically identical

@Bachmann1234 Bachmann1234 mentioned this pull request Nov 14, 2017
Merged
@coveralls
Copy link

coveralls commented Nov 14, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.05%) to 99.035% when pulling f4a4799 on replace_pep8_with_pycodestyle into e65a468 on master.

@coveralls
Copy link

coveralls commented Nov 14, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.05%) to 99.035% when pulling 406880b on replace_pep8_with_pycodestyle into e65a468 on master.

@Bachmann1234 Bachmann1234 force-pushed the replace_pep8_with_pycodestyle branch 3 times, most recently from cc94a90 to 08428d9 Compare November 14, 2017 03:17
@Bachmann1234
Python 2.6 broke. I am taking this as a chance to drop support for it
75efbba 
and python3.3 but will be supported pypy3 and python3.6
@coveralls
Copy link

coveralls commented Nov 14, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.0004%) to 99.081% when pulling 75efbba on replace_pep8_with_pycodestyle into e65a468 on master.

@Bachmann1234 Bachmann1234 merged commit 606147d into master Nov 14, 2017
@Bachmann1234 Bachmann1234 deleted the replace_pep8_with_pycodestyle branch November 14, 2017 03:33
@hugovk hugovk mentioned this pull request Dec 3, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Bachmann1234 @coveralls