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

Remove unnecessary modules #37

Merged
merged 1 commit into from Apr 5, 2017

Conversation

Projects
None yet
2 participants
@jjatria
Contributor

jjatria commented Apr 5, 2017

It turns out there were a lot of used modules and imported functions that were not used in the code.

I've removed all of the ones I could take out and still have all existing (unmodified) tests pass.

@oschwald

This comment has been minimized.

Show comment
Hide comment
@oschwald

oschwald Apr 5, 2017

Member

Nice cleanup! 👍

Member

oschwald commented Apr 5, 2017

Nice cleanup! 👍

@oschwald oschwald merged commit 3fdfd42 into maxmind:master Apr 5, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jjatria jjatria deleted the jjatria:remove-unused branch Apr 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment