Skip to content

Conversation

@bartgol
Copy link
Contributor

@bartgol bartgol commented Oct 27, 2025

Do not run these workflows when OTHER workflows are modified

[BFB]


@mahf708 I was tempted to use the same paths clause we use for ci also for ci-w, but I am not sure if the WCYCL compset is more broad than the ones in ci. Is that the case? If not, I can use the same paths clause (with the only differnece of the workflow file name at the bottom) to make it easier for who will maintain these.

@bartgol bartgol requested a review from mahf708 October 27, 2025 15:06
@bartgol bartgol self-assigned this Oct 27, 2025
@bartgol bartgol added BFB PR leaves answers BFB workflows labels Oct 27, 2025
Copy link
Contributor

@mahf708 mahf708 left a comment

Choose a reason for hiding this comment

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

ci-w is supposed to be a coupled case and so it is wider, but what you have is fine. These aren't that critical and I think --- THINK --- your edits are reasonable, so we should go with them for now

bartgol added a commit that referenced this pull request Oct 27, 2025
…7833)

Do not run these workflows when OTHER workflows are modified

[BFB]
@bartgol bartgol merged commit 6755ce5 into master Oct 27, 2025
8 checks passed
@bartgol bartgol deleted the bartgol/workflows/e3sm-gh-ci-minor-mods branch October 27, 2025 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BFB PR leaves answers BFB workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants