Skip to content

ci: adopt fleet-standard PR check workflows on main - #1314

Open
remko48 wants to merge 1 commit into
mainfrom
hotfix/workflow-sync
Open

ci: adopt fleet-standard PR check workflows on main#1314
remko48 wants to merge 1 commit into
mainfrom
hotfix/workflow-sync

Conversation

@remko48

@remko48 remko48 commented Aug 20, 2026

Copy link
Copy Markdown
Member

What

Syncs the PR check workflows from development to main, verbatim as they stand today:

  • Replaced/added: code-quality.yml, pull-request-lint-check.yaml, branch-protection.yml, merge-hygiene.yml — copied unchanged from development.
  • Removed: the legacy quality-check.yml, phpcs.yml, tests.yml, pull-request-quality-gate.yml, and pull-request-from-branch-check.yaml they superseded.

Why

#1113 aligned this repo's workflows with the fleet standard in ConductionNL/.github, but it merged into development only. PRs targeting main (every hotfix, e.g. #1313) still run the old inline quality workflow — a reduced check set on PHP 8.1 — instead of the shared quality suite, and don't report the check names the org ruleset expects.

Review notes

  • Files are byte-identical to their development counterparts (including workflow fixes that landed after ci: align workflows with fleet standard (ConductionNL/.github) #1113), so the upcoming developmentbetamain release merges resolve these files cleanly.
  • Release workflows (*-release.yaml, release.yml) are intentionally untouched: swapping release automation is out of scope for this fix, and leaving them unchanged on main also merges cleanly later since only development modified them.

@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/openconnector @ 42f834c

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
build
check-specs
test-l10n
format
composer ✅ 126/126
npm ✅ 569/569
app:check-code ⏭️
info.xml
REUSE
PHPUnit
Newman
Playwright
Hydra gates

Quality workflow — 2026-08-20 13:34 UTC

Download the full PDF report from the workflow artifacts.

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.

4 participants