Repository navigation
chore(ci): align cache pin documentation and retired path filter - #667
Merged
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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
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.
Problem and resolution
Preserve a previously committed local CI correction while consolidating work onto main. The setup action comment still names cache version 0.15.1 although its configured pin is 0.28.1, and the main push trigger includes the retired OpenWiki subtree.
Update the comment to the configured version and exclude
openwiki/**from push-triggered CI. Pull request triggers remain unchanged.Verification and risk
The diff contains two line additions and one line removal. The cache executable version is unchanged. Both files live in
.github, which the Docker build context excludes, so this correction does not change the deployed application bytes.Focused independent review found no issues. Local pre-push checks passed: actionlint, 32 changed-path tests, 81 workflow-shape tests, and structural checks. Exact-head hosted CI passed, including all three platform smokes and CodeQL configurations; workspace tests passed 6,587 with 9 skipped. Merged as
f2831126da7d0228874a4a4f2475f3a7942d6018. The application Docker context matches tested sourcede412c30a.