Skip to content
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

Skip some CIs if only Docs/ has changed #4197

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

WeiqunZhang
Copy link
Member

No description provided.

@WeiqunZhang
Copy link
Member Author

@EZoni This might work for WarpX.

@EZoni
Copy link

EZoni commented Oct 23, 2024

Thanks, @WeiqunZhang.

I guess we can try this on WarpX too, although, if really it isn't feasible to wait until the paths-ignore filter is fixed on the GitHub end, I was still hoping to come up with an "in-house" workaround, which we can control 100%, rather than relying on yet another external solution.

Reference for everyone, WarpX PR where we started this discussion: ECP-WarpX/WarpX#5387.

@WeiqunZhang
Copy link
Member Author

I was still hoping to come up with an "in-house" workaround, which we can control 100%, rather than relying on yet another external solution.

I don't think it's hard to do. But that's exactly why it's fine to use an external solution, because we can replace it.

@WeiqunZhang WeiqunZhang merged commit 8e7bb00 into AMReX-Codes:development Nov 8, 2024
71 checks passed
@WeiqunZhang WeiqunZhang deleted the skip_jobs branch November 8, 2024 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants