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.

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

[CI] Skip libcxx in non-llvm repo #73282

Merged
merged 1 commit into from
Nov 24, 2023
Merged

[CI] Skip libcxx in non-llvm repo #73282

merged 1 commit into from
Nov 24, 2023

Conversation

jsji
Copy link
Member

@jsji jsji commented Nov 24, 2023

The new libcxx workflow are run and failing in non llvm repo.
Skip it similar to other workflow.

The new libcxx workflow are run and failing in non llvm repo.
Skip it similar to other workflow.
@jsji jsji requested a review from EricWF November 24, 2023 04:23
@jsji jsji self-assigned this Nov 24, 2023
@llvmbot
Copy link
Collaborator

llvmbot commented Nov 24, 2023

@llvm/pr-subscribers-github-workflow

Author: Jinsong Ji (jsji)

Changes

The new libcxx workflow are run and failing in non llvm repo.
Skip it similar to other workflow.


Full diff: https://github.com/llvm/llvm-project/pull/73282.diff

1 Files Affected:

  • (modified) .github/workflows/libcxx-build-and-test.yaml (+3)
diff --git a/.github/workflows/libcxx-build-and-test.yaml b/.github/workflows/libcxx-build-and-test.yaml
index 905906a1c75c935..dccfe54d4646767 100644
--- a/.github/workflows/libcxx-build-and-test.yaml
+++ b/.github/workflows/libcxx-build-and-test.yaml
@@ -43,6 +43,7 @@ env:
 
 jobs:
   stage1:
+    if: github.repository_owner == 'llvm'
     runs-on:
       group: libcxx-runners-8
     continue-on-error: false
@@ -81,6 +82,7 @@ jobs:
             **/CMakeOutput.log
             **/crash_diagnostics/*
   stage2:
+    if: github.repository_owner == 'llvm'
     runs-on:
       group: libcxx-runners-8
     needs: [ stage1 ]
@@ -130,6 +132,7 @@ jobs:
             **/CMakeOutput.log
             **/crash_diagnostics/*
   stage3:
+    if: github.repository_owner == 'llvm'
     needs: [ stage1, stage2 ]
     continue-on-error: false
     strategy:

@tstellar
Copy link
Collaborator

@EricWF Do you test these in the libcxx org? If yes, then you might want to do a follow on commit that enables it for this org.

@jsji I think this is fine to commit now given the large impact it has on forks.

@jsji jsji merged commit 85ee351 into llvm:main Nov 24, 2023
37 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants