Skip to content

Commit

Permalink
Allow runing CI on external contributions (#5220) (#5289)
Browse files Browse the repository at this point in the history
* Refs #21479: Include labeling check

Signed-off-by: JesusPoderoso <[email protected]>

* Refs #21479: Keep 'Add label' job only in ubuntu workflow

Signed-off-by: JesusPoderoso <[email protected]>

* Refs #21479: Apply rev suggestions

Signed-off-by: eProsima <[email protected]>

---------

Signed-off-by: JesusPoderoso <[email protected]>
Signed-off-by: eProsima <[email protected]>
(cherry picked from commit 2d1e793)

Co-authored-by: Jesús Poderoso <[email protected]>
  • Loading branch information
mergify[bot] and JesusPoderoso authored Oct 3, 2024
1 parent e512de0 commit 478592d
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 33 deletions.
8 changes: 0 additions & 8 deletions .github/workflows/reusable-mac-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,14 +48,6 @@ jobs:
cmake-build-type:
- 'RelWithDebInfo'
steps:
- name: Add ci-pending label if PR
if: ${{ github.event_name == 'pull_request' }}
uses: eProsima/eProsima-CI/external/add_labels@v0
with:
labels: ci-pending
number: ${{ github.event.number }}
repo: eProsima/Fast-DDS

- name: Sync eProsima/Fast-DDS repository
uses: eProsima/eProsima-CI/external/checkout@v0
with:
Expand Down
16 changes: 0 additions & 16 deletions .github/workflows/reusable-sanitizers-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,14 +55,6 @@ jobs:
if: ${{ inputs.run_asan_fastdds == true }}
runs-on: ubuntu-22.04
steps:
- name: Add ci-pending label if PR
if: ${{ github.event_name == 'pull_request' }}
uses: eProsima/eProsima-CI/external/add_labels@v0
with:
labels: ci-pending
number: ${{ github.event.number }}
repo: eProsima/Fast-DDS

- name: Sync eProsima/Fast-DDS repository
uses: eProsima/eProsima-CI/external/checkout@v0
with:
Expand Down Expand Up @@ -342,14 +334,6 @@ jobs:
CC: gcc-12
CXX: g++-12
steps:
- name: Add ci-pending label if PR
if: ${{ github.event_name == 'pull_request' }}
uses: eProsima/eProsima-CI/external/add_labels@v0
with:
labels: ci-pending
number: ${{ github.event.number }}
repo: eProsima/Fast-DDS

- name: Sync eProsima/Fast-DDS repository
uses: eProsima/eProsima-CI/external/checkout@v0
with:
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/reusable-ubuntu-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,11 @@ on:
required: false
type: boolean
default: false
add-label:
description: 'Add the CI-PENDING label to the PR'
required: false
type: boolean
default: false

env:
security-cmake-flag: ${{ inputs.security == true && '-DSECURITY=ON' || '-DSECURITY=OFF' }}
Expand All @@ -71,7 +76,7 @@ jobs:
- 'RelWithDebInfo'
steps:
- name: Add ci-pending label if PR
if: ${{ github.event_name == 'pull_request' }}
if: ${{ github.event_name == 'pull_request' && inputs.add-label == true }}
uses: eProsima/eProsima-CI/external/add_labels@v0
with:
labels: ci-pending
Expand Down
8 changes: 0 additions & 8 deletions .github/workflows/reusable-windows-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,14 +40,6 @@ jobs:
- 'v141'
- 'v142'
steps:
- name: Add ci-pending label if PR
if: ${{ github.event_name == 'pull_request' }}
uses: eProsima/eProsima-CI/external/add_labels@v0
with:
labels: ci-pending
number: ${{ github.event.number }}
repo: eProsima/Fast-DDS

- name: Sync eProsima/Fast-DDS repository
uses: eProsima/eProsima-CI/external/checkout@v0
with:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/ubuntu-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,3 +69,4 @@ jobs:
run-build: ${{ !(github.event_name == 'pull_request') || !contains(github.event.pull_request.labels.*.name, 'skip-ci') }}
run-tests: ${{ ((inputs.run-tests == true) && true) || (github.event_name == 'pull_request' && !contains(github.event.pull_request.labels.*.name, 'no-test')) }}
use-ccache: ${{ ((inputs.use-ccache == true) && true) || false }}
add-label: ${{ (github.event_name == 'pull_request') && (github.event.pull_request.head.repo.full_name == github.repository) && true || false }}

0 comments on commit 478592d

Please sign in to comment.