Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

This allows for their reuse inside llvm-zorg. Otherwise we get an error that we cannot use reusable workflows inside job steps because Github thinks that they are reuseable workflows rather than composite actions. This should be NFC inside the monorepo.

This allows for their reuse inside llvm-zorg. Otherwise we get an error
that we cannot use reusable workflows inside job steps because Github
thinks that they are reuseable workflows rather than composite actions.
This should be NFC inside the monorepo.
@llvmbot
Copy link
Member

llvmbot commented Nov 6, 2025

@llvm/pr-subscribers-github-workflow

Author: Aiden Grossman (boomanaiden154)

Changes

This allows for their reuse inside llvm-zorg. Otherwise we get an error that we cannot use reusable workflows inside job steps because Github thinks that they are reuseable workflows rather than composite actions. This should be NFC inside the monorepo.


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

4 Files Affected:

  • (renamed) .github/actions/build-container/action.yml ()
  • (renamed) .github/actions/push-container/action.yml ()
  • (modified) .github/workflows/build-ci-container-tooling.yml (+4-4)
  • (modified) .github/workflows/build-ci-container.yml (+4-4)
diff --git a/.github/workflows/build-container/action.yml b/.github/actions/build-container/action.yml
similarity index 100%
rename from .github/workflows/build-container/action.yml
rename to .github/actions/build-container/action.yml
diff --git a/.github/workflows/push-container/action.yml b/.github/actions/push-container/action.yml
similarity index 100%
rename from .github/workflows/push-container/action.yml
rename to .github/actions/push-container/action.yml
diff --git a/.github/workflows/build-ci-container-tooling.yml b/.github/workflows/build-ci-container-tooling.yml
index 46dc38fe600a3..de32e1f9c5a8c 100644
--- a/.github/workflows/build-ci-container-tooling.yml
+++ b/.github/workflows/build-ci-container-tooling.yml
@@ -12,16 +12,16 @@ on:
       - '.github/workflows/containers/github-action-ci-tooling/**'
       - llvm/utils/git/requirements_formatting.txt
       - llvm/utils/git/requirements_linting.txt
-      - '.github/workflows/build-container/**'
-      - '.github/workflows/push-container/**'
+      - '.github/actions/build-container/**'
+      - '.github/actions/push-container/**'
   pull_request:
     paths:
       - .github/workflows/build-ci-container-tooling.yml
       - '.github/workflows/containers/github-action-ci-tooling/**'
       - llvm/utils/git/requirements_formatting.txt
       - llvm/utils/git/requirements_linting.txt
-      - '.github/workflows/build-container/**'
-      - '.github/workflows/push-container/**'
+      - '.github/actions/build-container/**'
+      - '.github/actions/push-container/**'
 
 jobs:
   build-ci-container-tooling:
diff --git a/.github/workflows/build-ci-container.yml b/.github/workflows/build-ci-container.yml
index 33b4dda2b2980..382d384d7ed3e 100644
--- a/.github/workflows/build-ci-container.yml
+++ b/.github/workflows/build-ci-container.yml
@@ -10,14 +10,14 @@ on:
     paths:
       - .github/workflows/build-ci-container.yml
       - '.github/workflows/containers/github-action-ci/**'
-      - '.github/workflows/build-container/**'
-      - '.github/workflows/push-container/**'
+      - '.github/actions/build-container/**'
+      - '.github/actions/push-container/**'
   pull_request:
     paths:
       - .github/workflows/build-ci-container.yml
       - '.github/workflows/containers/github-action-ci/**'
-      - '.github/workflows/build-container/**'
-      - '.github/workflows/push-container/**'
+      - '.github/actions/build-container/**'
+      - '.github/actions/push-container/**'
 
 jobs:
   build-ci-container:

@boomanaiden154 boomanaiden154 merged commit e30dc12 into llvm:main Nov 6, 2025
18 of 19 checks passed
@boomanaiden154 boomanaiden154 deleted the try-composite-actions-actually-actions branch November 6, 2025 23:25
vinay-deshmukh pushed a commit to vinay-deshmukh/llvm-project that referenced this pull request Nov 8, 2025
…lvm#166864)

This allows for their reuse inside llvm-zorg. Otherwise we get an error
that we cannot use reusable workflows inside job steps because Github
thinks that they are reuseable workflows rather than composite actions.
This should be NFC inside the monorepo.
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