-
-
Notifications
You must be signed in to change notification settings - Fork 120
Closed as not planned
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededstatus: pinnedShould not be labeled as staleShould not be labeled as staletype: docsDocumentation changesDocumentation changes
Description
The README mentions using a PAT when you want to re-trigger CI after adding a commit, but doesn’t mention the minimum permissions the PAT requires.
I created a PAT for add-and-commit to use, which I thought had enough permissions, but apparently I missed something, because it didn’t work.
Similarly, since the point of the PAT is to get around the don’t-rerun-checks guard that prevents build cycles, it would be helpful to have a note with some guidance on how to avoid check cycles (like what to put in an if on the add-and-commit step to ensure that it hasn’t already run on that PR, or maybe a conditional to set push to false or something).
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededstatus: pinnedShould not be labeled as staleShould not be labeled as staletype: docsDocumentation changesDocumentation changes