fix(ci): share the atomic rules pin across workflows - #697
Merged
Merged
Conversation
Karib0u
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI and release atomics declared their rules commit separately, allowing release binaries to use an older test suite. Store the commit once in
.github/atomic-rules-refand read it before each rules checkout in CI and release testing. Updating the suite now requires changing one file.Each reader requires a full commit SHA before passing it to checkout. Preserve the current
fca3fbf0ef1f1430f12f4f91bacb2205a0c0224frevision.Validation: parsed both workflow files and executed all three reader steps against valid, shortened, empty, multiline, and missing pins. Verified the exact checkout output for valid input and failure before output for invalid input.
git diff --checkpasses.