Skip to content

ci(lint): wire actionlint pre-commit hook and document workflow lint gate - #1743

Merged
lusoris merged 2 commits into
masterfrom
refactor/rc3-std-actionlint
Oct 1, 2026
Merged

lusoris merged 2 commits into
masterfrom
refactor/rc3-std-actionlint

Conversation

@lusoris

@lusoris lusoris commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Wire actionlint v1.7.12 into .pre-commit-config.yaml to enforce GitHub Actions workflow validity across all 35 workflow files under .github/workflows/ against .github/actionlint.yaml (HISS-11 hermetic supply chain pin). Add Makefile lint-actions target and document the actionlint gate in docs/development/pre-commit-hooks.md.

no state delta: measurement gap closing recorded audit debt; no defect row named

Deliverables

  • Research digest — no digest needed: wires standard linter matching repository convention
  • Decision matrix — no alternatives needed: follows established pre-commit and actionlint config
  • AGENTS.md invariant note — no rebase-sensitive changes needed: additive linter hook and documentation
  • Reproducer / smoke-test command — pre-commit run actionlint --all-files and make lint-actions
  • CHANGELOG fragment — changelog.d/added/std-actionlint-precommit.md
  • Rebase note — no rebase impact: additive linter hook and doc additions only

@lusoris
lusoris force-pushed the refactor/rc3-std-actionlint branch 2 times, most recently from 077e0cf to d430181 Compare October 1, 2026 17:49
…1142) (#1742)

* refactor(hip): bring hip tests part 2 to lint and HISS standard (ADR-1142)

Refactor the second batch of HIP test sources to eliminate clang-tidy debt
and satisfy HISS complexity limits:
- core/test/test_hip_wavefront_reduce.c
- core/test/test_hip_psnr_hvs_parity.c
- core/test/test_hip_speed_chroma_parity.c
- core/test/test_hip_speed_temporal_parity.c
- core/test/test_hip_speed_singular_parity.c
- core/test/test_hip_ciede_parity.c
- core/test/test_hip_motion_parity.c
- core/test/test_hip_vif_parity.c
- core/test/test_hip_psnr_parity.c
- core/test/test_hip_adm_parity.c

Functions exceeding statement and branch thresholds are decomposed into
focused helpers to conform with HISS-04 / NASA JPL Rule 4. Single-statement
loops and branches receive explicit braces. Portable C spelling of NULL
is preserved via ADR-1138 NOLINT brackets for MSVC compatibility.

Ratchet baselines in scripts/ci/tidy-baseline-*.json are tightened
(-156 warnings in the HIP lane down to 0 for all touched files, and -4
warnings across arm64, cpu, cuda, and sycl for wavefront reduce).
…gate (#1743)

* ci(lint): wire actionlint pre-commit hook and document workflow lint gate
@lusoris
lusoris force-pushed the refactor/rc3-std-actionlint branch from d430181 to 332b5ef Compare October 1, 2026 18:52
@lusoris
lusoris merged commit 332b5ef into master Oct 1, 2026
60 of 69 checks passed
@lusoris
lusoris deleted the refactor/rc3-std-actionlint branch October 1, 2026 18:52
@github-actions github-actions Bot added the type:ci CI and infrastructure label Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:ci CI and infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant