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

Use the official notation of the Django package #26

Merged
merged 1 commit into from Apr 13, 2017

Conversation

Projects
None yet
2 participants
@lociii
Contributor

lociii commented Apr 13, 2017

Even if pypi is case insensitive, all other packages include django with an uppercase D.
This package using lowercase will lead to uninstalls/reinstalls when using pip-compile and other tools.
Please accept the change to make it compatible.

Use the official notation of the Django package
Even if pypi is case insensitive, all other packages include django with an uppercase D.
This package using lowercase will lead to uninstalls/reinstalls when using pip-compile and other tools.
Please accept the change to make it compatible.

@kra3 kra3 merged commit de68ef5 into django-macaddress:master Apr 13, 2017

1 check passed

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

This comment has been minimized.

Show comment
Hide comment
@lociii

lociii Apr 14, 2017

Contributor

Thanks for merging the pull request.
Do you already know when an updated package will be released?

Contributor

lociii commented Apr 14, 2017

Thanks for merging the pull request.
Do you already know when an updated package will be released?

@kra3

This comment has been minimized.

Show comment
Hide comment
@kra3

kra3 Apr 15, 2017

Member
Member

kra3 commented Apr 15, 2017

@kra3

This comment has been minimized.

Show comment
Hide comment
@kra3

kra3 Apr 18, 2017

Member

@lociii I've release the new version earlier today. Have a look and let me know.
And Thanks for the PR.

Member

kra3 commented Apr 18, 2017

@lociii I've release the new version earlier today. Have a look and let me know.
And Thanks for the PR.

@lociii

This comment has been minimized.

Show comment
Hide comment
@lociii

lociii May 4, 2017

Contributor

Works as expected, fixed the issue on pip-compile. Thanks!

Contributor

lociii commented May 4, 2017

Works as expected, fixed the issue on pip-compile. Thanks!

@lociii lociii deleted the lociii:patch-1 branch May 4, 2017

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