- Notifications
You must be signed in to change notification settings - Fork455
-
I am using
I see error:
I notice a similar discussion around the PR request I am not sure whether the write permissions are ignored for This is a PR that comes from a fork and is being reviewed. I can add a label using the equivalent POST mentioned in the error using curl using my PAT. I tried adding my PAT as a secret in the repo and in an environment - but the secret PAT does not seem to be picked up and it says "Input required and not supplied: github-token". |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 1 comment 5 replies
-
If the pull request is from a fork, the permissions on the Actions |
BetaWas this translation helpful?Give feedback.
All reactions
-
@joshmgross thanks for your answer. A usual way we are working in Apache Flink is that everyone has a fork of Flink then they create a PR that will move the code from their fork to the master branch. I want to be able to add a label at review time of this PR in a github action. I can see stale github action polling all PRs and adding labels, but this polling and caching seems overkill when I want to do the action at review time. What options do you see for me to do this? |
BetaWas this translation helpful?Give feedback.
All reactions
-
|
BetaWas this translation helpful?Give feedback.
All reactions
-
@joshmgross thanks a lot for the pointer. it looks like pull_request_target only runs on main / master. I think the only option is to run a workflow that is a cron job - similar tohttps://github.com/actions/stale . Do you think I have any other options ? |
BetaWas this translation helpful?Give feedback.
All reactions
-
Is "review time" different from the time when a PR is opened? |
BetaWas this translation helpful?Give feedback.
All reactions
-
@joshmgross Yes I believe so, but ideally I would want to be able to list the PRs sorted by review time. I am not sure I can easily do that at the moment, maybe through graphql. I am thinking that I
WDYT? |
BetaWas this translation helpful?Give feedback.