Skip to content

[ci] Fix actionlint finding in reports workflow#245

Merged
coisa merged 4 commits intomainfrom
fix/244-actionlint-reports-read
Apr 24, 2026
Merged

[ci] Fix actionlint finding in reports workflow#245
coisa merged 4 commits intomainfrom
fix/244-actionlint-reports-read

Conversation

@coisa
Copy link
Copy Markdown
Contributor

@coisa coisa commented Apr 24, 2026

Related Issue

Closes #244

Motivation / Context

  • actionlint reported ShellCheck SC2162 in the reports workflow because the permission warning loop used read without -r.
  • The warning loop should preserve emitted paths exactly while keeping the existing GitHub annotation behavior.

Changes

  • Updates the reports workflow permission-fix loop to use while IFS= read -r line.
  • Publishes pending required PHPUnit statuses as soon as workflow-dispatched wiki pointer tests start.
  • Publishes each required PHPUnit status from its own matrix job instead of waiting for the aggregate matrix result.
  • Disables matrix fail-fast so one failing version does not leave another required status without a final state.
  • Adds changelog entries for the workflow lint and required-status fixes.

Verification

  • composer dev-tools
  • Focused command(s): actionlint .github/workflows/*.yml resources/github-actions/*.yml
  • Focused command(s): composer dev-tools changelog:check
  • Focused command(s): git diff --check
  • Focused command(s): validated the test-matrix PHP extraction snippet locally
  • Manual verification: Confirmed there is no equivalent permission loop in resources/github-actions/*.yml to align.

Documentation / Generated Output

  • README updated
  • docs/ updated
  • Generated or synchronized output reviewed

Changelog

  • Added a notable CHANGELOG.md entry

Reviewer Notes

  • Full composer dev-tools was not run because this PR changes workflow shell/status behavior only; the relevant workflow lint, changelog check, and matrix extraction smoke test passed locally.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2026

@coisa coisa merged commit 979cc87 into main Apr 24, 2026
32 checks passed
@coisa coisa deleted the fix/244-actionlint-reports-read branch April 24, 2026 19:10
@github-project-automation github-project-automation Bot moved this from Backlog to Released in PHP Fast Forward Project Apr 24, 2026
github-actions Bot added a commit that referenced this pull request Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working github-actions

Projects

Status: Released

Development

Successfully merging this pull request may close these issues.

Fix actionlint ShellCheck finding in reports workflow

1 participant