-
Notifications
You must be signed in to change notification settings - Fork 887
ci: handle retriggering ci and human authors in update-flake
#14052
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 1 | ||
token: ${{ secrets.CDRCI_GITHUB_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This secret is already available in Secrets. I assume that it has push/write access to the repo. We need to add/update a secret if it does not work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a PAT for the cdrci account with write perms, so it should be fine writing to PRs on this repo.
136ac14
to
d9cb961
Compare
✅ Meticulous spotted zero visual differences across 932 screens tested: view results. Expected differences? Click here. Last updated for commit c4cf764. This comment will update as new commits are pushed. |
d9cb961
to
c15cf47
Compare
Co-authored-by: Dean Sheather <dean@deansheather.com>
Co-authored-by: Dean Sheather <dean@deansheather.com>
This PR improves #14046 based on instructions here.
Should solve the CI stuck issue exhibited in