Skip to content

Conversation

@Marcono1234
Copy link
Contributor

@Marcono1234 Marcono1234 commented Oct 26, 2025

Closes #157

Based on the usage example in the README:
https://github.com/actions/first-interaction/tree/v3.1.0?tab=readme-ov-file#usage

For me the workflow was previously failing when I created my first issue in this repository here:
https://github.com/azagniotov/language-detection/actions/runs/18820093258/job/53694233157

...
Checking if its the users first contribution
Adding message: Thanks for opening your first issue. Pull requests are always welcome too! :) to issue 166
Error: Resource not accessible by integration - https://docs.github.com/rest/issues/comments#create-an-issue-comment

I am not completely sure if this change fully solves the problem for external contributors. Because normally pull_request runs with read-only permissions (which cannot be increased to write permission, e.g. pull-requests: write), unless GitHub added some special casing for actions/first-interaction?
But let's see once you approve the workflows for this PR whether the workflow is able to add a comment here.

@Marcono1234
Copy link
Contributor Author

Marcono1234 commented Oct 27, 2025

Sorry for the CI failure. Apparently the inputs names for actions/first-interaction were changed but their README had not been updated (see also actions/first-interaction#368).
Have changed the inputs names now.
Though unfortunately we will probably not see now whether this greetings workflow really works, until the next new contributor creates a PR.

Regarding the dependency-submission failure; probably that job should only run for push events, not for pull requests, right?

@azagniotov azagniotov force-pushed the greetings-workflow-fix branch from 178ed72 to b278f60 Compare October 31, 2025 02:02
@azagniotov azagniotov merged commit d001f4a into azagniotov:master Oct 31, 2025
7 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.

2 participants