Skip to content

Add baseline GitHub Actions checks#34

Open
jdevalk wants to merge 2 commits into
mainfrom
jdv/wp-github-actions-audit
Open

Add baseline GitHub Actions checks#34
jdevalk wants to merge 2 commits into
mainfrom
jdv/wp-github-actions-audit

Conversation

@jdevalk
Copy link
Copy Markdown
Member

@jdevalk jdevalk commented Mar 16, 2026

Summary

  • add WordPress coding standards checks
  • add PHP linting across supported PHP versions
  • add Composer diff and security audit workflows
  • add a Playground PR preview workflow

Why

This repo had no GitHub Actions coverage yet, so this adds a practical baseline without overengineering.

Open WordPress Playground Preview

jdevalk and others added 2 commits March 16, 2026 14:31
…l-lint)

The 'php-parallel-lint' tool name is not recognised by shivammathur/setup-php@v2;
the canonical name is 'parallel-lint'. This was causing all PHP lint matrix jobs
to fail with 'parallel-lint: command not found'.
@jdevalk jdevalk requested review from aristath and ilicfilip May 29, 2026 06:41
@ilicfilip
Copy link
Copy Markdown
Contributor

@jdevalk , we usually add Playgroud Preview button in a separate comment (example), so should we do it here as well for consitency?

It would also resolve the thing Claude complains about (I have not verified it):

1. playground-preview.yml triggers on edited — this can infinite-loop / spam.
The workflow uses mode: append-to-description, which edits the PR description. The trigger includes types: [..., edited]. A pull_request: edited event fires when the description changes — so the workflow's own description edit can re-trigger the workflow. The WordPress/action-wp-playground-pr-preview action uses <!-- wp-playground-preview:start/end --> markers (visible in this PR's body) to update-in-place rather than append repeatedly, which usually breaks the loop — but edited also fires on title edits and any manual body edit, causing needless re-runs. Recommend dropping edited from the trigger list unless there's a specific reason for it. At minimum, confirm the action is idempotent on its own edits (it appears to be, given the markers).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants