Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add gitter link in README #8190

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

Merged
merged 3 commits into from
Mar 6, 2017
Merged

Conversation

patniharshit
Copy link
Contributor

refs #8186

README.rst Outdated
The first is a good starting point for general questions and discussions.



.. image:: https://badges.gitter.im/Join%20Chat.svg
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a sentence that explains that gitter is for coordinating development and asking questions directly related to contributing to matplotlib?

@phobson phobson changed the title add gitter link in README [MRG+1] add gitter link in README Mar 4, 2017
README.rst Outdated


The gitter is for coordinating development and asking questions directly related to contributing to matplotlib.
Copy link
Member

@QuLogic QuLogic Mar 4, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"The gitter chat room" or just "Gitter", not "The gitter".

README.rst Outdated
The first is a good starting point for general questions and discussions.

.. image:: https://badges.gitter.im/Join%20Chat.svg
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we should repeat the image here; just have some text (like the alt text) pointing to the chat room.

@dstansby dstansby changed the title [MRG+1] add gitter link in README [MRG+2] add gitter link in README Mar 6, 2017
@phobson phobson merged commit 0a4c97f into matplotlib:master Mar 6, 2017
@phobson phobson changed the title [MRG+2] add gitter link in README add gitter link in README Mar 6, 2017
@phobson
Copy link
Member

phobson commented Mar 6, 2017

Thanks @patniharshit !

@QuLogic QuLogic added this to the 2.0.1 (next bug fix release) milestone Mar 6, 2017
@patniharshit patniharshit deleted the addGitterBack branch March 19, 2017 17:59
@dstansby
Copy link
Member

Is there any point backporting this since it only appears on github?

@QuLogic
Copy link
Member

QuLogic commented Mar 25, 2017

I think this corrects something that was already backported, but i don't feel super-strongly about it.

@dstansby dstansby modified the milestones: 2.1 (next point release), 2.0.1 (next bug fix release) Mar 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants