adding saved replies to development guide, under triage by noatamir · Pull Request #23109 · matplotlib/matplotlib · GitHub
Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 44 additions & 0 deletions doc/devel/git_workflows.rst
53 changes: 53 additions & 0 deletions doc/devel/saved_replies.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@

.. _saved_replies:

Saved replies
=============

To consistently maintain respectful and welcoming communication, reduce potential bias and errors, consistently provide
relevant information, and to avoid repitition, we decided to create a collection of saved replies for recurring project
needs. As a an active Matplotlib contributor, consider storing some of these responses in your GitHub account’s
`saved replies`_ as they might prove useful while reviewing issues and PRs, and otherwise supporting other community
members. Some of the suggested saved replies can be long and detailed. This is because we aim to offer support without
assuming the level of experience, or the intent the community member we are addressing might have. There is still room
to tweak and personalize your message if, for instance, some parts are redundant, require further information, or to
offer further support. Using these replies is optional.

.. _saved replies: https://github.com/settings/replies/

Issues
------

Coming soon 😉

Pull Requests
-------------

First Pull Request Merged

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just thinking: Could we have a bot write that message or is that too impersonal? We already have an automated welcome message (as you can see in this PR 😄).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think I would rather do them myself while I can manage the load. I would need to set up getting notifications for new PRs being merged. But I will consider it if it becomes unmanageable ☺️. It's between UX and building relationships, and feasibility.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have a slight preference for a human doing this. We do not have such a huge influx of new contributors and I think it does help build the inter-personal relationship.

^^^^^^^^^^^^^^^^^^^^^^^^^

.. code-block:: md

Hi-five ✋ on merging your first pull request to Matplotlib, @username! We hope you stick around and invite you to
continue to take an active part in Matplotlib! Your choices aren’t limited to programming 😉 – you can review pull
requests, help us stay on top of new and old issues, develop educational material, refresh our documentation, work
on our website, and so much more! For more info, check out: https://matplotlib.org/devdocs/devel/index

Did you know we have a monthly meeting 👥 for new contributors and a weekly meeting for the maintainers,
everyone is welcome to join both? You can find out more about our regular project meetings in this
[calendar page](https://scientific-python.org/calendars/).

Also, consider joining our [developer mailing list](https://mail.python.org/mailman/listinfo/matplotlib-devel) ✉️.
This is a great way to connect with other people in our community and be part of important conversations that
affect the development of Matplotlib.

Rebase
^^^^^^
.. code-block:: md

Hi, it looks your code needs to be rebased to include changes that have since occurred in the main repository. Would
you like to do this yourself, or would you like us to do this for you? I’m asking because a rebase can get a bit
fiddly and not everyone likes doing them 😉

We documented [the rebase process here](https://matplotlib.org/devdocs/devel/git_workflows#rebase.html) Have a look and
let me know how you'd like to proceed.
13 changes: 13 additions & 0 deletions doc/devel/triage.rst