Skip to content

Commit 0f354e6

Browse files
Improve GitHub Actions (openhab#4660)
* pin external action stCarolas/setup-maven to specific SHA * set global timeout * comments * restore broken registration of problem matchers Signed-off-by: Holger Friedrich <[email protected]>
1 parent 0b9a892 commit 0f354e6

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/ci-build.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ jobs:
2323
os: [ 'ubuntu-24.04' ]
2424
name: Build (Java ${{ matrix.java }}, ${{ matrix.os }})
2525
runs-on: ${{ matrix.os }}
26+
# typical duration is ~20min, set twice the amount as limit (default is 6h)
27+
timeout-minutes: 40
2628
steps:
2729
- name: Checkout
2830
if: github.head_ref == ''
@@ -50,13 +52,14 @@ jobs:
5052
distribution: 'temurin'
5153
java-version: ${{ matrix.java }}
5254

55+
# pinning to SHA to mitigate possible supply chain attacks
5356
- name: Set up Maven ${{ matrix.maven }}
54-
uses: stCarolas/setup-maven@v5
57+
uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
5558
with:
5659
maven-version: ${{ matrix.maven }}
5760

5861
- name: Register Problem Matchers
59-
if: ${{ matrix.java == '17' }}
62+
if: ${{ matrix.java == '21' }}
6063
id: problem_matchers
6164
run: |
6265
echo "::add-matcher::.github/openhab-compile-problems.json"
@@ -93,6 +96,8 @@ jobs:
9396
path: '**/checkstyle-result.xml'
9497
mode: inline
9598

99+
# this will identify modifications to files under source control during the workflow run;
100+
# untracked files will be included as well!
96101
- name: Verify Changed Files
97102
id: verify-changed-files
98103
run: |

0 commit comments

Comments
 (0)