Skip to content

Commit 5da24e1

Browse files
amadioapeters1971
authored andcommitted
[CI] Update ABI compliance check build
- Move permissions down to job level - Update to actions/checkout@v6 and actions/upload-artifact@v7 - Fix trigger rule to build whenever changes are made within src/ - Use git describe to find better match for closest base ref tag
1 parent be7404a commit 5da24e1

1 file changed

Lines changed: 11 additions & 7 deletions

File tree

.github/workflows/ABI.yml

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,18 @@
11
name: ABI
22

3-
permissions:
4-
contents: read
5-
63
on:
74
push:
5+
branches:
6+
- '**'
87
paths:
98
- .github/workflows/ABI.yml
109
- 'src/**'
11-
branches-ignore:
12-
- skip-ci
13-
tags-ignore:
1410
pull_request:
11+
branches:
12+
- '**'
13+
paths:
14+
- .github/workflows/ABI.yml
15+
- 'src/**'
1516
workflow_dispatch:
1617

1718
concurrency:
@@ -32,6 +33,9 @@ jobs:
3233
name: ABI Compatibility Check
3334
runs-on: ubuntu-latest
3435

36+
permissions:
37+
contents: read
38+
3539
steps:
3640
- name: Install dependencies
3741
run: |
@@ -58,7 +62,7 @@ jobs:
5862
run: cmake --build build/new --parallel --target install
5963

6064
- name: Checkout Base Reference of XRootD to Compare ABI
61-
run: git checkout -f ${GITHUB_BASE_REF:-$(git merge-base $(git rev-list --tags --max-count=1) HEAD)}
65+
run: git checkout -f ${GITHUB_BASE_REF:-$(git describe --abbrev=0 --always HEAD)}
6266

6367
- name: Configure XRootD (Base Reference)
6468
run: cmake -S . -B build/old -DCMAKE_INSTALL_PREFIX=install/old

0 commit comments

Comments
 (0)