Skip to content

Conversation

@bobrippling
Copy link
Collaborator

@bobrippling bobrippling commented Sep 15, 2025

Testing #3994

  • Move PR action to a scheduled job
  • In the scheduled job:
    • Generate a list of PRs that haven't already been commented on by this bot
    • Tag in app-authors on those PRs (using the existing tag script)

@bobrippling
Copy link
Collaborator Author

bobrippling commented Sep 15, 2025

From the run

Changed app: apps/rep, author field: bobrippling

🎉

Error: GitHub API error: 403

😭

I think to have this get permissions (from GITHUB_TOKEN) to comment on PRs, it can't run as a job on the PR itself. It'll have to be a scheduled job which checks for new PRs and comments on them. I'll tinker with that but it'll be a few weeks

I've disabled the pr-tagging workflow for the time being

@RKBoss6
Copy link
Contributor

RKBoss6 commented Dec 15, 2025

How's this going so far?

@bobrippling
Copy link
Collaborator Author

Stalled at the moment, I've updated the todos in the initial comment to clarify what I reckon the next steps are

@RKBoss6
Copy link
Contributor

RKBoss6 commented Dec 16, 2025

Hmm, just to clarify, what's stopping you from getting the token?

@bobrippling
Copy link
Collaborator Author

bobrippling commented Dec 16, 2025

Time and (my) prioritisation of this issue :)

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