Skip to content

Add breakage CI#258

Merged
nHackel merged 8 commits into
masterfrom
nh/breakage
Aug 4, 2025
Merged

Add breakage CI#258
nHackel merged 8 commits into
masterfrom
nh/breakage

Conversation

@nHackel

@nHackel nHackel commented Aug 1, 2025

Copy link
Copy Markdown
Member

This PR adds breakage CI jobs, which check if changes to package X caused issues in the tests of downstream packages Y.

In our case if MRIBase caused breaks in any of the other packages and if MRIOperators caused issues in MRIReco and MRISimulations

@nHackel

nHackel commented Aug 2, 2025

Copy link
Copy Markdown
Member Author

This would just be redundant with the MRIReco_All workflow unless I figure out a way to checkout the latest release version of a package. I only know how to do this for packages with tags.

To fix my issues with the yaml file I have to register the matrix entry as an ENV and use that in the shell/run commands

@nHackel nHackel closed this Aug 2, 2025
@nHackel nHackel deleted the nh/breakage branch August 2, 2025 14:34
@nHackel nHackel restored the nh/breakage branch August 4, 2025 07:30
@nHackel nHackel reopened this Aug 4, 2025
@nHackel nHackel changed the title Try adding breakage CI Add breakage CI Aug 4, 2025
@nHackel nHackel merged commit 675f8e0 into master Aug 4, 2025
26 of 28 checks passed
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.

1 participant