From 109ac30f704a9ab0c840304caa21386a571496dd Mon Sep 17 00:00:00 2001 From: "Michael A. Smith" Date: Mon, 11 May 2026 15:47:50 -0400 Subject: [PATCH 1/3] ci: invoke require-checker via composer script 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. --- .github/workflows/composer-require-checker.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/composer-require-checker.yml b/.github/workflows/composer-require-checker.yml index 3794789..36678be 100644 --- a/.github/workflows/composer-require-checker.yml +++ b/.github/workflows/composer-require-checker.yml @@ -25,5 +25,5 @@ jobs: uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 with: name: Run Composer Require Checker - run: composer-require-checker check --config-file=.composer-require-checker.json + run: composer require-checker php-tools: 'composer:v2, composer-require-checker' From 46df6704d7f58d5ec4af63a9a76e56a9cb359436 Mon Sep 17 00:00:00 2001 From: "Michael A. Smith" Date: Tue, 12 May 2026 09:20:59 -0400 Subject: [PATCH 2/3] ci: broaden paths filters that were too narrow - 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. --- .github/workflows/php-syntax-check.yml | 2 ++ .github/workflows/phpcs.yml | 4 ++++ .github/workflows/tests.yml | 14 -------------- 3 files changed, 6 insertions(+), 14 deletions(-) diff --git a/.github/workflows/php-syntax-check.yml b/.github/workflows/php-syntax-check.yml index 9629a0a..44c315c 100644 --- a/.github/workflows/php-syntax-check.yml +++ b/.github/workflows/php-syntax-check.yml @@ -6,12 +6,14 @@ on: - main paths: - '**.php' + - composer.json - .github/workflows/php-syntax-check.yml pull_request: branches: - main paths: - '**.php' + - composer.json - .github/workflows/php-syntax-check.yml jobs: diff --git a/.github/workflows/phpcs.yml b/.github/workflows/phpcs.yml index fd94abd..669b0f8 100644 --- a/.github/workflows/phpcs.yml +++ b/.github/workflows/phpcs.yml @@ -8,6 +8,8 @@ on: - '**.php' - .phpcs.xml* - phpcs.xml* + - composer.json + - composer.lock - .github/workflows/phpcs.yml pull_request: branches: @@ -16,6 +18,8 @@ on: - '**.php' - .phpcs.xml* - phpcs.xml* + - composer.json + - composer.lock - .github/workflows/phpcs.yml jobs: diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 0528dd6..850478f 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -4,23 +4,9 @@ on: push: branches: - main - paths: - - '**.php' - - composer.json - - composer.lock - - phpunit.xml - - phpunit.xml.dist - - .github/workflows/tests.yml pull_request: branches: - main - paths: - - '**.php' - - composer.json - - composer.lock - - phpunit.xml - - phpunit.xml.dist - - .github/workflows/tests.yml jobs: test: From b211ea5595776ca1b42fe6a633442efb19778861 Mon Sep 17 00:00:00 2001 From: "Michael A. Smith" Date: Tue, 12 May 2026 10:15:36 -0400 Subject: [PATCH 3/3] ci: bump reusable workflows pin to 0.0.4 --- .github/workflows/actionlint.yml | 2 +- .github/workflows/composer-normalize.yml | 2 +- .github/workflows/composer-require-checker.yml | 2 +- .github/workflows/composer-validate.yml | 2 +- .github/workflows/conventional-pr-title.yml | 2 +- .github/workflows/php-syntax-check.yml | 2 +- .github/workflows/phpcs.yml | 2 +- .github/workflows/phpstan.yml | 2 +- .github/workflows/rector.yml | 2 +- .github/workflows/tests.yml | 2 +- 10 files changed, 10 insertions(+), 10 deletions(-) diff --git a/.github/workflows/actionlint.yml b/.github/workflows/actionlint.yml index a4ed69d..7235a2e 100644 --- a/.github/workflows/actionlint.yml +++ b/.github/workflows/actionlint.yml @@ -14,4 +14,4 @@ on: jobs: actionlint: - uses: openCoreEMR/github-workflows-public/.github/workflows/actionlint.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/actionlint.yml@0.0.4 diff --git a/.github/workflows/composer-normalize.yml b/.github/workflows/composer-normalize.yml index 838c41f..c5b1afd 100644 --- a/.github/workflows/composer-normalize.yml +++ b/.github/workflows/composer-normalize.yml @@ -16,7 +16,7 @@ on: jobs: composer-normalize: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: Normalize composer.json (check) run: composer normalize --dry-run diff --git a/.github/workflows/composer-require-checker.yml b/.github/workflows/composer-require-checker.yml index 36678be..23d8cfd 100644 --- a/.github/workflows/composer-require-checker.yml +++ b/.github/workflows/composer-require-checker.yml @@ -22,7 +22,7 @@ on: jobs: composer-require-checker: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: Run Composer Require Checker run: composer require-checker diff --git a/.github/workflows/composer-validate.yml b/.github/workflows/composer-validate.yml index 42a41c9..2ebf3ac 100644 --- a/.github/workflows/composer-validate.yml +++ b/.github/workflows/composer-validate.yml @@ -16,7 +16,7 @@ on: jobs: composer-validate: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: Validate composer.json run: composer validate --strict diff --git a/.github/workflows/conventional-pr-title.yml b/.github/workflows/conventional-pr-title.yml index 6a08eec..c1cf897 100644 --- a/.github/workflows/conventional-pr-title.yml +++ b/.github/workflows/conventional-pr-title.yml @@ -6,4 +6,4 @@ on: jobs: conventional-pr-title: - uses: openCoreEMR/github-workflows-public/.github/workflows/conventional-pr-title.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/conventional-pr-title.yml@0.0.4 diff --git a/.github/workflows/php-syntax-check.yml b/.github/workflows/php-syntax-check.yml index 44c315c..5fba220 100644 --- a/.github/workflows/php-syntax-check.yml +++ b/.github/workflows/php-syntax-check.yml @@ -18,7 +18,7 @@ on: jobs: php-syntax-check: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: PHP Syntax Check run: composer php-lint diff --git a/.github/workflows/phpcs.yml b/.github/workflows/phpcs.yml index 669b0f8..9ebe6e5 100644 --- a/.github/workflows/phpcs.yml +++ b/.github/workflows/phpcs.yml @@ -24,7 +24,7 @@ on: jobs: phpcs: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: Run PHP CodeSniffer run: composer phpcs diff --git a/.github/workflows/phpstan.yml b/.github/workflows/phpstan.yml index 50d1bb3..4dc6ae4 100644 --- a/.github/workflows/phpstan.yml +++ b/.github/workflows/phpstan.yml @@ -26,7 +26,7 @@ on: jobs: phpstan: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: Run PHPStan run: composer phpstan diff --git a/.github/workflows/rector.yml b/.github/workflows/rector.yml index b050bd5..d6dce3b 100644 --- a/.github/workflows/rector.yml +++ b/.github/workflows/rector.yml @@ -22,7 +22,7 @@ on: jobs: rector: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@0.0.4 with: name: Run Rector (dry-run) run: composer rector diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 850478f..9bee533 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -10,4 +10,4 @@ on: jobs: test: - uses: openCoreEMR/github-workflows-public/.github/workflows/php-tests.yml@0.0.2 + uses: openCoreEMR/github-workflows-public/.github/workflows/php-tests.yml@0.0.4