Skip to content

fix(ci): stabilize checks and remove duplicate Alpine coverage - #1464

Merged
Chemaclass merged 5 commits into
mainfrom
fix/windows-ci-results
Nov 8, 2026
Merged

Chemaclass merged 5 commits into
mainfrom
fix/windows-ci-results

Conversation

@Chemaclass

@Chemaclass Chemaclass commented Nov 8, 2026 •

Copy link
Copy Markdown
Member

Background

CI failed in Windows summary tests, reported unsupported permission tests as risky, and exited before existing network skip checks in strict mode. Release asset downloads and duplicate Alpine sequential coverage also delayed or failed CI.

Changes

  • Initialize summary test state directly and report unsupported directory permission tests as skipped.
  • Let existing download and binary checks classify command failures in strict mode.
  • Fetch pinned ShellCheck through the GitHub API, run make sa, and retain .bash coverage while excluding the intentional syntax-error fixture.
  • Keep both Alpine parallel modes, remove the duplicate sequential job, and use the existing offline flag for Alpine tests. Sequential execution and live download tests remain on other platforms.

Validation: focused parallel and strict tests, forced download failure cases, ShellCheck, lint, YAML parsing, and review passed. The offline install and upgrade tests reported 19 passes and 12 skips. All 37 CI checks passed on the final commit. Alpine extended finished in 1m 56s and simple in 3m 9s; the duplicate sequential job was removed.

Checklist

  • Changelog reviewed: contributor test and CI changes only, no public behavior change
  • Documentation reviewed: no public API or usage change

@Chemaclass Chemaclass added the bug Something isn't working label Nov 8, 2026
@Chemaclass Chemaclass self-assigned this Nov 8, 2026
@Chemaclass
Chemaclass enabled auto-merge November 8, 2026 10:47
@Chemaclass
Chemaclass disabled auto-merge November 8, 2026 10:53

@Chemaclass Chemaclass left a comment •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review found one ShellCheck coverage regression.

Comment thread .github/workflows/static_analysis.yml Resolved
@Chemaclass Chemaclass changed the title test(ci): isolate summary state and mark skips fix(ci): stabilize checks and remove duplicate Alpine coverage Nov 8, 2026
@Chemaclass
Chemaclass enabled auto-merge November 8, 2026 11:21
@Chemaclass
Chemaclass disabled auto-merge November 8, 2026 11:22
@Chemaclass
Chemaclass merged commit da4fb2b into main Nov 8, 2026
38 checks passed
@Chemaclass
Chemaclass deleted the fix/windows-ci-results branch November 8, 2026 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant