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

Minor codestyle fixes #55

Merged
merged 2 commits into from Oct 12, 2017

Conversation

Projects
None yet
2 participants
@mre
Contributor

mre commented Oct 11, 2017

These are just some minor adjustments that don't change the logic of the library.
Mostly guided by clippy.

@m4b

m4b approved these changes Oct 12, 2017

This is awesome @mre thanks for this!

@m4b

This comment has been minimized.

Show comment
Hide comment
@m4b

m4b Oct 12, 2017

Owner

Heh I dunno wth is going on with GitHub today. Says I can’t merge cause of a server error and it’s out of sync with base branch (sure but so what there aren’t any conflicts afaics)

Owner

m4b commented Oct 12, 2017

Heh I dunno wth is going on with GitHub today. Says I can’t merge cause of a server error and it’s out of sync with base branch (sure but so what there aren’t any conflicts afaics)

@m4b m4b merged commit 0ac82c7 into m4b:master Oct 12, 2017

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment