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
7 changes: 4 additions & 3 deletions .github/actions/complete-pr-comment/action.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
name: "Complete PR Comment"
description:
"Removes the 'eyes' reaction and adds a completion reaction ('rocket' for success, 'confused' for 'failure', 'cancelled', or 'skipped') to the
triggering comment."
description: >-
Removes the 'eyes' reaction and adds a completion reaction ('rocket' for success, 'confused' for 'failure',
'cancelled', or 'skipped') to the triggering comment. Requires `issues: write` permission in the calling
workflow to update comment reactions via the GitHub Reactions API.
inputs:
status:
description:
Expand Down
4 changes: 4 additions & 0 deletions .github/actions/workflow-setup/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ inputs:
exclude-globs:
description: "Exclude globs for change detection"
required: false
head-ref:
description: "Explicit head ref for change detection (overrides the resolved ref)"
required: false

outputs:
is_act:
Expand Down Expand Up @@ -119,3 +122,4 @@ runs:
file-type: ${{ inputs.file-type }}
include-globs: ${{ inputs.include-globs }}
exclude-globs: ${{ inputs.exclude-globs }}
head-ref: ${{ inputs.head-ref }}
7 changes: 6 additions & 1 deletion .github/workflows/actionlint-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,11 +62,16 @@ jobs:
.github/workflows/**/*.yaml
.github/actions/**/*.yml
.github/actions/**/*.yaml
head-ref: ${{ inputs.pr-head-sha }}

actionlint-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-latest
permissions:
contents: read
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/clang-format-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
clang-format-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (github.event_name == 'workflow_dispatch' || needs.setup.outputs.has_changes == 'true')
runs-on: ubuntu-latest
permissions:
contents: read
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/clang-format-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ run-name: "${{ github.actor }} fixing C++ code format"
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/clang-tidy-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
clang-tidy-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (github.event_name == 'workflow_dispatch' || needs.setup.outputs.has_changes == 'true')
runs-on: ubuntu-24.04
container:
image: ghcr.io/framework-r-d/phlex-ci:latest
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/cmake-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,7 @@ jobs:
pr-base-sha: ${{ inputs.pr-base-sha }}
checkout-path: ${{ inputs.checkout-path }}
build-path: ${{ inputs.build-path }}
head-ref: ${{ inputs.pr-head-sha }}
file-type: |
cpp
cmake
Expand All @@ -123,7 +124,11 @@ jobs:
build:
needs: [setup, generate-matrix]
if: >
always() && needs.setup.result == 'success' && needs.setup.outputs.has_changes == 'true'
always() && needs.setup.result == 'success' && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-24.04
strategy:
fail-fast: false
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/cmake-format-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,16 @@ jobs:
uses: Framework-R-D/phlex/.github/actions/workflow-setup@main
with:
file-type: cmake
head-ref: ${{ inputs.pr-head-sha }}

cmake-format-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-latest
steps:
- name: Checkout code
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/cmake-format-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ run-name: "${{ github.actor }} fixing CMake format"
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/coverage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ jobs:
coverage:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (github.event_name == 'workflow_dispatch' || needs.setup.outputs.has_changes == 'true')
runs-on: ubuntu-24.04
container:
image: ghcr.io/framework-r-d/phlex-ci:latest
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/header-guards-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,16 @@ jobs:
uses: Framework-R-D/phlex/.github/actions/workflow-setup@main
with:
file-type: cpp
head-ref: ${{ inputs.pr-head-sha }}

header-guards-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-latest
permissions:
contents: read
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/header-guards-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ run-name: "${{ github.actor }} fixing header guards"
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/jsonnet-format-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,16 @@ jobs:
uses: Framework-R-D/phlex/.github/actions/workflow-setup@main
with:
file-type: jsonnet
head-ref: ${{ inputs.pr-head-sha }}

jsonnet-format-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-latest
container:
image: public.ecr.aws/bitnami/jsonnet:latest
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/jsonnet-format-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ run-name: "${{ github.actor }} fixing Jsonnet format"
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/markdown-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,16 @@ jobs:
uses: Framework-R-D/phlex/.github/actions/workflow-setup@main
with:
file-type: md
head-ref: ${{ inputs.pr-head-sha }}

markdown-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-latest
permissions:
contents: read
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/markdown-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ run-name: "${{ github.actor }} fixing Markdown format"
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/python-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,16 @@ jobs:
uses: Framework-R-D/phlex/.github/actions/workflow-setup@main
with:
file-type: python
head-ref: ${{ inputs.pr-head-sha }}

python-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (
github.event_name == 'workflow_dispatch' ||
inputs.skip-relevance-check ||
needs.setup.outputs.has_changes == 'true'
)
runs-on: ubuntu-latest
permissions:
contents: read
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/python-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ run-name: "${{ github.actor }} fixing Python code"
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/yaml-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
yaml-check:
needs: setup
if: >
always() && needs.setup.outputs.has_changes == 'true'
always() && (github.event_name == 'workflow_dispatch' || needs.setup.outputs.has_changes == 'true')
runs-on: ubuntu-latest
steps:
- name: Checkout code
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/yaml-fix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ name: YAML Fix
permissions:
pull-requests: write
contents: write
issues: write

jobs:
setup:
Expand Down
Loading