Skip to content

Conversation

@reduxionist
Copy link
Contributor

django-chartjs_engine -> django-chartjs-engine

Tiniest commit ever, but it did break the link to the project from PyPI so I thought you'd want to know -- but feel free to ignore this PR and just roll the change into whenever you next edit setup.py if you prefer... 😄

Thanks for sharing your code!

django-chartjs_engine -> django-chartjs-engine
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.788% when pulling 968a928 on reduxionist:patch-1 into ea29875 on deltaskelta:master.

@jeffwillette jeffwillette merged commit 2658d78 into jeffwillette:master Feb 13, 2017
@jeffwillette
Copy link
Owner

Thanks, It was already broken since the underscore was a typo. BTW, this was the first PR I've ever gotten so thanks!

@reduxionist reduxionist deleted the patch-1 branch February 13, 2017 08:14
@reduxionist
Copy link
Contributor Author

Oh cool, glad to have made the first of what I'm sure will be many PR's in time -- even if it will always be the smallest one! 😀

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.

3 participants