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

WIP: apply some pylint throughout #43

Merged
merged 1 commit into from
Oct 13, 2017
Merged

WIP: apply some pylint throughout #43

merged 1 commit into from
Oct 13, 2017

Conversation

zramsay
Copy link
Collaborator

@zramsay zramsay commented Oct 13, 2017

@djrtwo
Copy link
Collaborator

djrtwo commented Oct 13, 2017

👍
merging

@djrtwo djrtwo merged commit d97e007 into blockchain Oct 13, 2017
@djrtwo djrtwo deleted the apply-some-pylint branch October 13, 2017 14:50
@zramsay zramsay changed the title apply some pylint throughout WIP: apply some pylint throughout Oct 13, 2017
@djrtwo
Copy link
Collaborator

djrtwo commented Oct 13, 2017

whoops...

@zramsay
Copy link
Collaborator Author

zramsay commented Oct 13, 2017

ha no worries, i'll keep adding

@djrtwo
Copy link
Collaborator

djrtwo commented Oct 13, 2017

just saw your WIP tag.
Jumped the gun on that...

@djrtwo
Copy link
Collaborator

djrtwo commented Oct 13, 2017

lol 👍

@zramsay
Copy link
Collaborator Author

zramsay commented Oct 13, 2017

i like the enthusiasm ...

@djrtwo
Copy link
Collaborator

djrtwo commented Oct 13, 2017

i was just making a new branch off blockchain and thought "I'd like these changes in my new branch"

@zramsay
Copy link
Collaborator Author

zramsay commented Oct 13, 2017

no harm done, i was late on the WIP b/c i decided to keep plugging away at the lints

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants