build(deps): bump ci-workflows pin to b6431a1 (PSSA RULE_ERROR retry fix) - #44
Merged
Merged
Conversation
…fix) Picks up melodic-software/ci-workflows#41 (b6431a1644f2): the PSScriptAnalyzer runner now retries any RULE_ERROR crash, including the CommandNotFoundException ("Get-Command is not recognized") variant of the #1708 runspace race that intermittently failed the powershell lane. No rules disabled; full SCA; Linux. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
kyle-sexton
added a commit
that referenced
this pull request
Jul 7, 2026
## Problem Both scheduled online link-check runs behind the rolling tracking issues did not fail on dead links — every one of this repo's 173 external links resolves (verified with a full-repo online lychee 0.24.2 run against `modules/lychee/lychee.toml`). The runs died before checking anything: `error: unexpected argument '--accept-timeouts' found`. The reusable ci-workflows `link-check.yml` passes default args that rely on `--accept-timeouts` (lychee ≥ 0.24) while leaving lychee-action at its default **lychee v0.23.0**, which also predates the `include_fragments = "full"` config form this repo's ruleset requires (module engine floor 0.24.2+). ## Change 1. **Fixture exclusion (this branch, ready):** the online lane also verifies local file links, so once the version is fixed it would flag `fixtures/lychee/bad/` on every scheduled run and re-open the tracking issue forever. The caller now overrides the called workflow's `args` to add `--exclude-path 'fixtures/[^/]+/bad'`, mirroring the offline lane's exclusion in `ci.yml` (`args` is a single input, so the defaults are restated). Verified locally: with the exclusion, a full online run reports 0 errors. 2. **Pin bump (pending, this PR stays draft until then):** the version fix itself lives in the reusable workflow — melodic-software/ci-workflows#52 (pins `lycheeVersion: v0.24.2` and moves lychee-action past v2.8.0 for the 0.24.x archive layout). That PR is green/CLEAN but not yet merged. Once it lands, this branch gets the repo-wide ci-workflows pin bump to the new SHA (per the convention from #44, which also refreshes the stale `# b6431a1` pin comments left by the Dependabot SHA-only bump) and the PR will be marked ready. ## Verification plan After merge: `workflow_dispatch` the link-check workflow and confirm a green end-to-end run. Closes #13 Closes #54 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_017K9RRMdUKD7HLExmHwMXAe Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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.
Bumps the pinned
melodic-software/ci-workflowsSHA tob6431a1to pick up melodic-software/ci-workflows#41 — the PSScriptAnalyzer runner now retries anyRULE_ERRORcrash, covering theCommandNotFoundException("Get-Command is not recognized") variant of the #1708 race that intermittently failed thepowershelllane. No rules disabled; full SCA; Linux runners.🤖 Generated with Claude Code