Skip to content

Commit

Permalink
Harmonize workflow branch filters
Browse files Browse the repository at this point in the history
  • Loading branch information
pavetok committed Sep 9, 2023
1 parent cd2d4f9 commit 0bfc848
Show file tree
Hide file tree
Showing 5 changed files with 56 additions and 18 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/pipeline-beyond-doubt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ name: Pipeline beyond doubt

on:
pull_request:
branches: [main]
branches:
- main
workflow_dispatch:

jobs:
Expand Down
15 changes: 12 additions & 3 deletions .github/workflows/pipeline-sanity.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ name: Pipeline sanity

on:
pull_request:
branches-ignore: [main]
branches-ignore:
- main
paths: # waiting for https://github.com/actions/runner/issues/2324
- .dx/**
- .tools/**
Expand All @@ -12,8 +13,8 @@ on:
workflow_dispatch:

jobs:
sources:
name: Sources in ${{ matrix.environ.name }} environ
up-to-binaries:
name: Up to binaries in ${{ matrix.environ.name }} environ
timeout-minutes: 5
strategy:
matrix:
Expand All @@ -34,3 +35,11 @@ jobs:
env:
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- run: >
ansible-playbook binaries.yml
-e binary_repo=maven.pkg.github.com
-e focus=pipeline
working-directory: .dx
env:
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
3 changes: 2 additions & 1 deletion .github/workflows/solution-beyond-doubt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ name: Solution beyond doubt

on:
merge_group:
branches: [main]
branches:
- main
workflow_dispatch:

env:
Expand Down
16 changes: 6 additions & 10 deletions .github/workflows/solution-convincing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,13 @@
name: Solution convincing

on:
merge_group:
branches:
- feature/**
- bugfix/**
pull_request:
branches: [main]
branches:
- main
workflow_dispatch:

jobs:
Expand Down Expand Up @@ -91,12 +96,3 @@ jobs:
update-comment: true
env:
JACOCO_INTEGRATIONS: .tools/target/coverage/integrations/jacoco.xml
- uses: madrapps/[email protected]
if: ${{ github.event_name == 'pull_request' && hashFiles(env.JACOCO_OVERALL) != '' }}
with:
title: Overall Coverage Report
paths: ${{ env.JACOCO_OVERALL }}
token: ${{ secrets.GITHUB_TOKEN }}
update-comment: true
env:
JACOCO_OVERALL: .tools/target/coverage/overall/jacoco.xml
37 changes: 34 additions & 3 deletions .github/workflows/solution-sanity.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ name: Solution sanity

on:
pull_request:
branches-ignore: [main]
branches-ignore:
- main
paths: # waiting for https://github.com/actions/runner/issues/2324
- libs/**
- apps/**
Expand All @@ -12,8 +13,8 @@ on:
workflow_dispatch:

jobs:
sources:
name: Sources
up-to-binaries:
name: Up to binaries
timeout-minutes: 5
runs-on: ubuntu-22.04
steps:
Expand All @@ -26,3 +27,33 @@ jobs:
env:
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- run: >
ansible-playbook binaries.yml
-e binary_repo=maven.pkg.github.com
-e focus=solution
working-directory: .dx
env:
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- uses: EnricoMi/[email protected]
if: ${{ !cancelled() }}
with:
check_name: Unit Test Report
check_run_annotations: none
compare_to_earlier_commit: false
action_fail_on_inconclusive: true
action_fail: true
job_summary: false
comment_mode: off
files: |
libs/*/target/surefire-reports/TEST-*.xml
apps/*/target/surefire-reports/TEST-*.xml
- uses: madrapps/[email protected]
if: ${{ github.event_name == 'pull_request' && hashFiles(env.JACOCO_UNITS) != '' }}
with:
title: Unit Coverage Report
paths: ${{ env.JACOCO_UNITS }}
token: ${{ secrets.GITHUB_TOKEN }}
update-comment: true
env:
JACOCO_UNITS: .tools/target/coverage/units/jacoco.xml

0 comments on commit 0bfc848

Please sign in to comment.