Skip to content

MAINT: Move to first-contribution action#31090

Merged
QuLogic merged 1 commit intomatplotlib:mainfrom
melissawm:fix-bot
Feb 6, 2026
Merged

MAINT: Move to first-contribution action#31090
QuLogic merged 1 commit intomatplotlib:mainfrom
melissawm:fix-bot

Conversation

@melissawm
Copy link
Member

PR summary

Supersedes #30635
Related to #31043 (one will need a rebase after the other is merged)

Fixes the "PR Greeting" workflow by moving to the https://github.com/plbstl/first-contribution action as suggested by @bjlittle .

This action will:

  • Post a comment to a first-contributor's PR, just like it did before.
  • Add a "first-contributor" label to issues opened by users who had never opened issues in matplotlib/matplotlib before.

There is an option to add a message to first-time issues, but I'm not sure we need that. Also open to reformulate the first -time PR message if that's needed. One thing I had in mind is that maybe we want to point folks to the Discourse forum instead of Gitter?

Tests

I tested this on my fork. You can see the results here:

PR checklist

Co-authored-by: Ruth Comer <10599679+rcomer@users.noreply.github.com>
@bjlittle
Copy link

bjlittle commented Feb 5, 2026

@melissawm If it helps, here's my GHA workflow over on geovista:
https://github.com/bjlittle/geovista/blob/main/.github/workflows/ci-first-contribution.yml

An example first-issue interaction is here:

bjlittle/geovista#2021

I really like the level of control that it offers 👌

HTH 🍻

Copy link
Member

@story645 story645 left a comment

Choose a reason for hiding this comment

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

Cool! Can it also add a first-contribution label to PRs for easier filtering?

@story645
Copy link
Member

story645 commented Feb 5, 2026

Also the deal w/ gitter/discourse is that we were waiting on seeing if discourse would get adopted before updating the docs to point to it.

@QuLogic
Copy link
Member

QuLogic commented Feb 6, 2026

These seems like one of those things we just have to merge and see if it works...

@QuLogic QuLogic merged commit d68c7e3 into matplotlib:main Feb 6, 2026
32 of 37 checks passed
@QuLogic QuLogic added this to the v3.11.0 milestone Feb 6, 2026
@rcomer
Copy link
Member

rcomer commented Feb 6, 2026

I think someone with admin rights might also need to turn it back on in the GitHub interface.

@QuLogic
Copy link
Member

QuLogic commented Feb 6, 2026

Already did so.

@rcomer
Copy link
Member

rcomer commented Feb 10, 2026

Hmmm this seems wrong

Archiljain does not meet the criteria for being a first timer. Exiting..

https://github.com/matplotlib/matplotlib/actions/runs/21857753226/job/63078581562
#31128

@rcomer
Copy link
Member

rcomer commented Feb 11, 2026

It lives! #31133 (comment)

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.

5 participants