- Notifications
You must be signed in to change notification settings - Fork465
-
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 8 replies
-
If the pull request is from a fork, the permissions on the Actions |
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.
All reactions
👍 1
-
That definitely sounds feasible, let me know how it goes! |
BetaWas this translation helpful?Give feedback.
All reactions
-
Hi@joshmgross , unfortunately the cache in the above case gets written to refs/pull/7/merge. So is not in a branch, as such the cron github action does not have access to it! I think the only way I can do this is to write a cron job that does a graphql for all open prs with reviews, we have less than 200 - as we use the stalebot to keep the numbers down. Then look into each PR and see whether a recent review has been done (within the cron period ish) and then process those to add the label. I really wanted to use a trigger in some way to limit what we need to process, but it seems there is no way to do this currently with forked repos. If I am missing something, that would allow me to run with a triggered approach - that would be great to hear. |
BetaWas this translation helpful?Give feedback.
All reactions
-
I am going to see if artefacts have the same restrictions as the cache as well. |
BetaWas this translation helpful?Give feedback.