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

a slightly opinionated autopep8 run #870

Merged
merged 1 commit into from
Oct 30, 2018
Merged

a slightly opinionated autopep8 run #870

merged 1 commit into from
Oct 30, 2018

Conversation

gromgull
Copy link
Member

@gromgull gromgull commented Oct 30, 2018 •
edited
Loading

opinions is mainly: no to long lines, but not at any cost.

notation3.py crashses autopep :D

Also rdflib/__init__.py gets completely broken

@gromgull gromgull added this to the rdflib 5.0.0 milestone Oct 30, 2018
@gromgull gromgull force-pushed the autopep8 branch 2 times, most recently from 5d80ef6 to 5825b87 Compare October 30, 2018 14:26
@gromgull
a slightly opinionated autopep8 run
785e379 
opinions is mainly: no to long lines, but not at any cost.

notation3.py crashses autopep :D

Also rdflib/__init__.py gets completely broken
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 75.622% when pulling 785e379 on autopep8 into f4c706f on master.

@gromgull gromgull merged commit f2d8931 into master Oct 30, 2018
@gromgull gromgull deleted the autopep8 branch October 30, 2018 14:44
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
cleanup
Projects
None yet
Milestone
rdflib 5.0.0
Development

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