Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.1k
MNT: add dummy issue message to PR greeting workflow#30635
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
rcomer commentedOct 5, 2025
I am somewhat suspicious that it thinks that was my first PR in my fork. So opened a couple more and it commented on those too. I note that in the action's own repo the dependabot PRs are getting flagged for the new contributor messageactions/first-interaction#363 (comment) Something seems off to me. |
rcomer commentedOct 15, 2025
The problem that this PR was trying to fix is now tracked atactions/first-interaction#365. The problem of every PR being identified as a new contribution is now tracked atactions/first-interaction#369. So I think it's better to wait and see what happens over there. |
heinrich5991 commentedOct 17, 2025
Off-topic: You can reopen a closed pull request by first force-pushing the expected commit hash, reopening and then force-pushing the wanted commit hash again:https://gist.github.com/robertpainsi/2c42c15f1ce6dab03a0675348edd4e2c. |
ksunden commentedOct 28, 2025
I have disabled the workflow (via Github Web UI) in the mean time until the problems are addressed (As it is not doing what we intend and is giving CI failures where they are not warranted) |
PR summary
Replacement for#30633, since if you force push when the PR is closed, you can't re-open.
Hopefully fixes the PR greeting action, which currently fails likethis. I had read the error message wrong 🤦♀️, and the
${name}variable in the code was actuallyissue_messagein our case. Here is an example in my fork of the workflow now running:rcomer#5 (comment). I also tried settingissue_messageto an empty string,but that didn't work.PR checklist