Skip to content

Add issue checker - #3511

Merged
robertclaus merged 9 commits into
devfrom
add-issue-check
Nov 21, 2025
Merged

Add issue checker#3511
robertclaus merged 9 commits into
devfrom
add-issue-check

Conversation

@ndrezn

@ndrezn ndrezn commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

Closes #3508

Primarily to help the core team review incoming community PRs and keep track of changes.

GHA can be found in: https://github.com/plotly/pr-issue-checker

@ndrezn
ndrezn requested a review from robertclaus November 13, 2025 21:00
@github-actions

github-actions Bot commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Dash! 🎉

Comment thread .github/workflows/pr-issue-check.yml Outdated
Comment thread .github/workflows/pr-issue-check.yml
Comment thread .github/workflows/pr-issue-check.yml Outdated
Comment thread .github/workflows/pr-issue-check.yml Outdated
@AnnMarieW

AnnMarieW commented Nov 13, 2025

Copy link
Copy Markdown
Collaborator

What does the error message say if there is no linked issue? Lots of people don’t know you have to use a magic word before the issue number like fixes or closes. And it needs to be in the first issue message block, otherwise GH doesn’t automatically close the issue when it’s merged.

Comment thread .github/workflows/pr-issue-check.yml
Comment thread .github/workflows/pr-issue-check.yml Outdated
Comment thread .github/workflows/pr-issue-check.yml
Comment thread .github/workflows/pr-issue-check.yml Outdated
@ndrezn

ndrezn commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

@AnnMarieW the failure message looks like this:
image

@camdecoster camdecoster left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There might be some kinks to work out, but it seems like we should turn it on and see how it goes.

@ndrezn

ndrezn commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

@T4rk1n could I get a ✅ ?

@ndrezn
ndrezn requested a review from emilykl November 17, 2025 18:04
@robertclaus
robertclaus merged commit 30afe78 into dev Nov 21, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Require that every open PR be associated with an issue

6 participants