Skip to content

Fix workflow-dispatched test runs#258

Merged
coisa merged 3 commits intomainfrom
fix/tests-workflow-dispatch-inputs
Apr 24, 2026
Merged

Fix workflow-dispatched test runs#258
coisa merged 3 commits intomainfrom
fix/tests-workflow-dispatch-inputs

Conversation

@coisa
Copy link
Copy Markdown
Contributor

@coisa coisa commented Apr 24, 2026

Summary

  • treat max-outdated as a string workflow input so manual dispatch validation does not fail before jobs are created
  • stop passing explicit max-outdated=-1 from release, wiki, and conflict automation dispatches, relying on the tests workflow default instead
  • keep packaged consumer test wrappers aligned with the reusable workflow input contract
  • teach pull-request publication guidance to read back the published body and fix literal escaped Markdown control characters before handoff

Verification

  • actionlint .github/workflows/tests.yml resources/github-actions/tests.yml .github/workflows/changelog.yml .github/workflows/wiki-preview.yml
  • ruby -e 'require "yaml"; ARGV.each { |file| YAML.load_file(file); puts file }' .github/workflows/tests.yml resources/github-actions/tests.yml .github/workflows/changelog.yml .github/workflows/wiki-preview.yml
  • composer dev-tools changelog:check
  • git diff --check
  • GrumPHP pre-commit hook
  • gh workflow run tests.yml --repo php-fast-forward/dev-tools --ref fix/tests-workflow-dispatch-inputs -f publish-required-statuses=true -> run 24915679472 completed successfully with matrix jobs and required status publication

Follow-up from #250.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2026

@coisa coisa merged commit 6cb1b52 into main Apr 24, 2026
32 checks passed
@coisa coisa deleted the fix/tests-workflow-dispatch-inputs branch April 24, 2026 23:08
@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

None yet

Projects

Status: Released

Development

Successfully merging this pull request may close these issues.

1 participant