Skip to content

ci: migrate to reusable PHP CI workflows#14

Merged
kojiromike merged 5 commits into
mainfrom
kojiromike/use-reusable-workflows
May 12, 2026
Merged

ci: migrate to reusable PHP CI workflows#14
kojiromike merged 5 commits into
mainfrom
kojiromike/use-reusable-workflows

Conversation

@kojiromike

@kojiromike kojiromike commented May 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Migration to the reusable PHP CI workflows in openCoreEMR/github-workflows-public@0.0.2. Replaces the inlined workflows with thin callers.

Same pattern as the pilot: openCoreEMR/oce-lib-module-config#13. This repo also uses the dclint and hadolint reusables.

This module is PHP 8.5-only, so the test matrix and coverage-php-version are pinned to 8.5 (and every php-composer-script caller passes php-version: 8.5).

Test plan

  • All reusable-backed checks run and pass on PHP 8.5

Replaces inlined workflows with thin callers to
openCoreEMR/github-workflows-public@0.0.2.
@kojiromike kojiromike force-pushed the kojiromike/use-reusable-workflows branch from 0908ca9 to 9797fa4 Compare May 11, 2026 12:54
Calls 'composer require-checker' (defined in composer.json) instead of
inlining the underlying binary command, so CI uses the same script
developers run locally and the command details stay in one place.
- phpcs, php-syntax-check: add composer.json (and composer.lock for
  phpcs) since the linters are invoked via composer scripts
- tests: drop paths filter; tests depend on too many runtime assets
  (templates, SQL, config) to enumerate safely

Caught by Copilot on oce-module-sinch-conversations#132.
@kojiromike kojiromike merged commit 60b8646 into main May 12, 2026
11 checks passed
@kojiromike kojiromike deleted the kojiromike/use-reusable-workflows branch May 12, 2026 14:27
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.

1 participant