Skip to content

MAINT: Move to first-contribution action#31090

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

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

Conversation

@melissawm

Copy link
Copy Markdown
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

@bjlittle

bjlittle commented Feb 5, 2026

Copy link
Copy Markdown

@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 🍻

@story645 story645 left a comment

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.

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

@story645

story645 commented Feb 5, 2026

Copy link
Copy Markdown
Member

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

QuLogic commented Feb 6, 2026

Copy link
Copy Markdown
Member

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

rcomer commented Feb 6, 2026

Copy link
Copy Markdown
Member

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

@QuLogic

QuLogic commented Feb 6, 2026

Copy link
Copy Markdown
Member

Already did so.

@rcomer

rcomer commented Feb 10, 2026

Copy link
Copy Markdown
Member

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

rcomer commented Feb 11, 2026

Copy link
Copy Markdown
Member

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