Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ci] build with clingtest flag active if interpreter folder was touched #16917

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

[ci] build with clingtest flag active if interpreter folder was touched #16917

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fc561a9
[ci] build with clingtest flag active if interpreter folder was touched
ferdymercury Nov 13, 2024
d5f3ef3
[ci] fix stray condition
ferdymercury Nov 13, 2024
97c3373
Merge branch 'master' into clingtest
ferdymercury Nov 19, 2024
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
27 changes: 27 additions & 0 deletions .github/workflows/root-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,29 @@ concurrency:
cancel-in-progress: true

jobs:
conditional_job_check_files:
# https://www.meziantou.net/executing-github-actions-jobs-or-steps-only-when-specific-files-change.htm
runs-on: ubuntu-latest
# Declare outputs for next jobs
outputs:
interpreter_changed: ${{ steps.check_file_changed.outputs.interpreter_changed }}
steps:
- uses: actions/checkout@v2
with:
# Checkout as many commits as needed for the diff
fetch-depth: 2
- shell: pwsh
id: check_file_changed
run: |
# Diff HEAD with the previous commit
$diff = git diff --name-only HEAD^ HEAD

# Check if a file under interpreter/ has changed
$SourceDiff = $diff | Where-Object { $_ -match '^interpreter/' }
$HasDiff = $SourceDiff.Length -gt 0

# Set the output named "interpreter_changed"
Write-Host "::set-output name=interpreter_changed::$HasDiff"
build-macos:
# For any event that is not a PR, the CI will always run. In PRs, the CI
# can be skipped if the tag [skip-ci] or [skip ci] is written in the title.
Expand Down Expand Up @@ -340,6 +363,7 @@ jobs:
build-linux:
# For any event that is not a PR, the CI will always run. In PRs, the CI
# can be skipped if the tag [skip-ci] or [skip ci] is written in the title.
needs: [ conditional_job_check_files ]
if: |
(github.repository_owner == 'root-project' && github.event_name != 'pull_request') ||
(github.event_name == 'pull_request' && !(contains(github.event.pull_request.title, '[skip-ci]') || contains(github.event.pull_request.title, '[skip ci]')))
Expand All @@ -364,6 +388,9 @@ jobs:
overrides: ["LLVM_ENABLE_ASSERTIONS=On"]
- image: alma9
overrides: ["LLVM_ENABLE_ASSERTIONS=On", "CMAKE_BUILD_TYPE=Debug"]
- image: alma9-clingtest
if: needs.conditional_job_check_files.outputs.interpreter_changed == 'True'
overrides: ["LLVM_ENABLE_ASSERTIONS=On", "CMAKE_BUILD_TYPE=Debug", "clingtest=ON]
- image: ubuntu22
overrides: ["imt=Off", "LLVM_ENABLE_ASSERTIONS=On", "CMAKE_BUILD_TYPE=Debug"]
- image: ubuntu2404
Expand Down