Skip to content

Add @andrewdnolan to CI reviewers lists#910

Merged
xylar merged 2 commits into
MPAS-Dev:mainfrom
altheaden:small-ci-updates
Aug 8, 2025
Merged

Add @andrewdnolan to CI reviewers lists#910
xylar merged 2 commits into
MPAS-Dev:mainfrom
altheaden:small-ci-updates

Conversation

@altheaden
Copy link
Copy Markdown
Collaborator

@altheaden altheaden commented Aug 7, 2025

This PR simply adds @andrewdnolan to various lists we have for reviewing CI.

@xylar Review checklist:

@altheaden altheaden added the ci Changes affect Azure Pipelines CI label Aug 7, 2025
@altheaden altheaden requested a review from xylar August 7, 2025 17:49
@altheaden
Copy link
Copy Markdown
Collaborator Author

@xylar I am not sure if we need to do those same CI fixes over here on Compass as we've been doing in the other repos. I've left them out for now just because it seems pre-commit is updated over here much less often (I assume just because we use ruff elsewhere but not here). So, maybe we can add them here if needed once we get things sorted out elsewhere. Or, I can add them now in this PR - up to you.

Copy link
Copy Markdown
Collaborator

@xylar xylar left a comment

Choose a reason for hiding this comment

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

@xylar xylar merged commit 6e84bde into MPAS-Dev:main Aug 8, 2025
6 checks passed
@xylar xylar deleted the small-ci-updates branch August 8, 2025 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes affect Azure Pipelines CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants