Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 3 additions & 7 deletions .github/workflows/actionlint.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
# actionlint reusable workflow.
#
# Recommended caller stanza:
# Recommended caller stanza. Do not add `paths:` to `pull_request`; it
# breaks promotion to a required check (skipped run = missing check =
# auto-merge stall). Path filtering on `push` is fine.
#
# name: actionlint
# on:
Expand All @@ -11,9 +13,6 @@
# - .github/actions/**
# pull_request:
# branches: [main]
# paths:
# - .github/workflows/**
# - .github/actions/**
# jobs:
# actionlint:
# uses: openCoreEMR/github-workflows-public/.github/workflows/actionlint.yml@<tag>
Expand All @@ -31,9 +30,6 @@ on:
pull_request:
branches:
- main
paths:
- .github/workflows/**
- .github/actions/**
workflow_call:
inputs:
runs-on:
Expand Down
10 changes: 3 additions & 7 deletions .github/workflows/dclint.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
# dclint (Docker Compose linter) reusable workflow.
#
# Recommended caller stanza:
# Recommended caller stanza. Do not add `paths:` to `pull_request`; it
# breaks promotion to a required check (skipped run = missing check =
# auto-merge stall). Path filtering on `push` is fine.
#
# name: dclint
# on:
Expand All @@ -14,12 +16,6 @@
# - .github/workflows/dclint.yml
# pull_request:
# branches: [main]
# paths:
# - '**/compose.yml'
# - '**/compose.yaml'
# - '**/docker-compose*.yml'
# - '**/docker-compose*.yaml'
# - .github/workflows/dclint.yml
# jobs:
# dclint:
# uses: openCoreEMR/github-workflows-public/.github/workflows/dclint.yml@<tag>
Expand Down
7 changes: 3 additions & 4 deletions .github/workflows/hadolint.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
# hadolint (Dockerfile linter) reusable workflow.
#
# Recommended caller stanza:
# Recommended caller stanza. Do not add `paths:` to `pull_request`; it
# breaks promotion to a required check (skipped run = missing check =
# auto-merge stall). Path filtering on `push` is fine.
#
# name: hadolint
# on:
Expand All @@ -11,9 +13,6 @@
# - .github/workflows/hadolint.yml
# pull_request:
# branches: [main]
# paths:
# - '**/Dockerfile*'
# - .github/workflows/hadolint.yml
# jobs:
# hadolint:
# uses: openCoreEMR/github-workflows-public/.github/workflows/hadolint.yml@<tag>
Expand Down
20 changes: 8 additions & 12 deletions .github/workflows/php-composer-script.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,10 @@
# Generic reusable for any "composer install + composer <script>" job
# (phpstan, phpcs, rector, composer-validate, composer-normalize,
# composer-require-checker, php-syntax-check, etc.). The caller should
# scope the trigger to relevant paths so the job only runs when needed.
# scope the `push` trigger to relevant paths so post-merge runs on
# `main` only fire when needed. Do NOT add `paths:` to `pull_request`;
# it breaks promotion to a required check (skipped run = missing check
# = auto-merge stall).
#
# Recommended caller stanza (PHPStan):
#
Expand All @@ -19,14 +22,6 @@
# - .github/workflows/phpstan.yml
# pull_request:
# branches: [main]
# paths:
# - '**.php'
# - phpstan.neon
# - phpstan.neon.dist
# - phpstan-baseline.neon
# - composer.json
# - composer.lock
# - .github/workflows/phpstan.yml
# jobs:
# phpstan:
# uses: openCoreEMR/github-workflows-public/.github/workflows/php-composer-script.yml@<tag>
Expand All @@ -36,9 +31,10 @@
# name: Run PHPStan
# run: composer phpstan
#
# Recommended path filters by script. Always include composer.json (and
# composer.lock for anything that runs `composer install`) since each job
# is invoked through a composer script that's defined there:
# Recommended `push.paths` filters by script. Always include
# composer.json (and composer.lock for anything that runs
# `composer install`) since each job is invoked through a composer
# script that's defined there:
#
# composer-validate composer.json
# composer-normalize composer.json
Expand Down