build(workflows): remove stale Node.js version matrix includes in linux_test_install#13231
Draft
Planeshifter wants to merge 1 commit into
Draft
build(workflows): remove stale Node.js version matrix includes in linux_test_install#13231Planeshifter wants to merge 1 commit into
linux_test_install#13231Planeshifter wants to merge 1 commit into
Conversation
…nux_test_install`
The `test_install` job in `linux_test_install.yml` failed on every
nightly run on develop for 30+ consecutive days with the workflow
reporting an overall failure while zero `test_install` jobs actually
ran (only the unrelated Zulip notification job executed).
Root cause: `NODE_VERSION` was trimmed to `['20', '18', '16']`, but
the `include` list still listed entries for Node.js 14, 12, 10, 8, 6,
4, 0.12, and 0.10. Since those versions no longer match any base
matrix combination, GitHub Actions adds each as a standalone job
configuration missing `OS`, `BUILD_TASK`, and `PACKAGE_MANAGER`,
which breaks `runs-on: ${{ matrix.OS }}` and invalidates the entire
job's matrix expansion. This commit comments out the stale include
entries, matching the pattern already used in the sibling
`linux_test.yml` and `macos_test.yml` workflows.
Ref: https://github.com/stdlib-js/stdlib/actions/runs/28581985462
Draft
7 tasks
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.
Description
This pull request:
includeentries from thetest_installjob's build matrix inlinux_test_install.yml.Failing run: https://github.com/stdlib-js/stdlib/actions/runs/28581985462
Symptom:
linux_test_installhas failed on every nightlydeveloprun for 30+ consecutive days. The run reports overallconclusion: failure, but zerotest_installjobs actually run — only the unrelatedZulip notificationjob executes.Root cause:
NODE_VERSIONwas trimmed to['20', '18', '16'], butincludestill lists entries for14,12,10,8,6,4,0.12,0.10. None of those values match any base matrix combination, so GitHub Actions adds each as a standalone job configuration containing only the keys given in theincludeentry — missingOS,BUILD_TASK, andPACKAGE_MANAGER.runs-on: ${{ matrix.OS }}then resolves to an empty string, invalidating the job's entire matrix expansion, so notest_installjobs run and the workflow reports failure.Fix: comment out the eight stale
includeentries, identical to the pattern already in use in the siblinglinux_test.ymlandmacos_test.ymlworkflows.Related Issues
This pull request has no related issues.
Questions
No.
Other
Validation: YAML syntax checked with
python3 -c "import yaml; yaml.safe_load(...)". Three independent agent reviews (correctness, regression scope, style) all approved with no blocking findings, confirming the remainingincludeentries all match existing base combinations and that the comment style matcheslinux_test.yml/macos_test.ymlcharacter-for-character.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This pull request was prepared by Claude Code as part of an automated CI-failure investigation routine. Root cause was identified from job logs and GitHub Actions matrix semantics, cross-checked against the identical pattern already applied in sibling workflow files, and validated by three independent agent reviews.
@stdlib-js/reviewers
Generated by Claude Code