Skip to content

Switch pre-commit update workflow to use two tokens - #141

Merged
xylar merged 1 commit into
MPAS-Dev:mainfrom
xylar:fix-ci-again
Aug 13, 2025
Merged

Switch pre-commit update workflow to use two tokens#141
xylar merged 1 commit into
MPAS-Dev:mainfrom
xylar:fix-ci-again

Conversation

@xylar

@xylar xylar commented Aug 6, 2025

Copy link
Copy Markdown
Collaborator

In this merge, we go back to the simplified workflow that does not use conda to install gh and instead use the latest gh from github-actions.

This requires using a personal access token (PAT) to set reviewers and labels, so one has been added to the repo.

We still use the github-actions bot to open the PR so it doesn't get opened as @xylar (and @xylar can still review).

@xylar
xylar requested a review from altheaden August 6, 2025 12:59
@xylar xylar self-assigned this Aug 6, 2025
@xylar xylar added bug Something isn't working ci Changes that affect github actions CI labels Aug 6, 2025
@xylar
xylar force-pushed the fix-ci-again branch 2 times, most recently from f89c29e to 5610edc Compare August 6, 2025 13:04
In this merge, we go back to the simplified workflow that does
not use conda to install `gh` and instead use the latest `gh`
from github-actions.

This requires using a personal access token (PAT) to set reviewers
and labels, so one has been added to the repo.

We still use the github-actions bot to open the PR so it doesn't
get opened as @xylar (and @xylar can still review).
@xylar

xylar commented Aug 8, 2025

Copy link
Copy Markdown
Collaborator Author

@altheaden, if you could have a look at this one and MPAS-Dev/compass#912, we could then port this to the other "owned" repos.

@altheaden altheaden left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xylar Sorry for the delay in getting to this. That all looks good to me. I'll try to get this ported to a few more repos so that we can see how things go tomorrow.

@xylar
xylar merged commit d6eec24 into MPAS-Dev:main Aug 13, 2025
6 checks passed
@xylar
xylar deleted the fix-ci-again branch August 13, 2025 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci Changes that affect github actions CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants