-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Addresses #8177, Readme badges #8178
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
Conversation
I would suggest a plain text gitter link next to the mailing-list section. You can also update the pypi link so that it points directly to pypi rather than via a redirection. PS for whoever is reviewing this: you can check how it looks like at https://github.com/epintozzi/matplotlib/tree/readme_badges. |
Can we be vain and not include codecov? 😈 Agree on including text link to gitter with the mailing lists the 'view' button in the review page will also show you the rendering. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor preference for a text link to gitter, but fine with this going in as-is.
Please backport to 2.0.0-doc
I'll consider that a positive review @tacaswell, and merge! |
This looks like your first contribution to Matplotlib @epintozzi, thanks a lot! |
Addresses #8177, Readme badges
Backported to |
Added badges for: Travis, Appveyor, Codecov, pypi, gitter, and depsy