Skip to content

Commit 877d404

Browse files
committed
ci: Use files to skip extra steps
1 parent da64fce commit 877d404

File tree

1 file changed

+11
-21
lines changed

1 file changed

+11
-21
lines changed

Diff for: .github/workflows/lint.yml

+11-21
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,11 @@ jobs:
5050
name: nf-core-changes
5151
runs-on: ubuntu-latest
5252
outputs:
53-
tags: ${{ steps.filter.outputs.changes }}
54-
modules: ${{ steps.tags.outputs.modules }}
55-
subworkflows: ${{ steps.tags.outputs.subworkflows }}
53+
# https://github.com/dorny/paths-filter?tab=readme-ov-file#custom-processing-of-changed-files
54+
modules: ${{ steps.filter.outputs.modules }}
55+
modules_files: ${{ steps.filter.outputs.modules_files }}
56+
subworkflows: ${{ steps.filter.outputs.subworkflows}}
57+
subworkflows_files: ${{ steps.filter.outputs.subworkflows_files }}
5658
steps:
5759
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
5860
with:
@@ -71,32 +73,20 @@ jobs:
7173
token: ""
7274
list-files: "json"
7375

74-
- name: Fetch module tags
75-
id: module-tags
76-
if: steps.filter.outputs.modules == 'true'
77-
run: |
78-
echo "modules=${{ steps.filter.outputs.modules }}" >> $GITHUB_OUTPUT
79-
80-
- name: Fetch subworkflows tags
81-
id: subworkflow-tags
82-
if: steps.filter.outputs.subworkflows == 'true'
83-
run: |
84-
echo "subworkflows=${{ steps.filter.outputs.subworkflows }}" >> $GITHUB_OUTPUT
85-
8676
- name: debug
8777
run: |
88-
echo ${{ steps.module-tags.outputs.modules }}
89-
echo ${{ steps.subworkflow-tags.outputs.subworkflows }}
78+
echo ${{ steps.filter.outputs.modules_files }}
79+
echo ${{ steps.filter.outputs.subworkflows_files }}
9080
9181
nf-core-lint-modules:
9282
runs-on: ${{ github.event.inputs.runners || 'self-hosted' }}
9383
name: nf-core lint modules
9484
needs: nf-core-changes
95-
if: ${{ (needs.nf-core-changes.outputs.modules != '[]') }}
85+
if: ${{ (needs.nf-core-changes.outputs.modules == 'true') }}
9686
strategy:
9787
fail-fast: false
9888
matrix:
99-
tags: "${{ fromJson(needs.nf-core-changes.outputs.modules) }}"
89+
tags: "${{ fromJson(needs.nf-core-changes.outputs.modules_files) }}"
10090
steps:
10191
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
10292

@@ -134,11 +124,11 @@ jobs:
134124
runs-on: ubuntu-latest
135125
name: nf-core lint subworkflows
136126
needs: nf-core-changes
137-
if: ${{ (needs.nf-core-changes.outputs.subworkflows != '[]') }}
127+
if: ${{ (needs.nf-core-changes.outputs.subworkflows != 'true') }}
138128
strategy:
139129
fail-fast: false
140130
matrix:
141-
tags: "${{ fromJson(needs.nf-core-changes.outputs.subworkflows) }}"
131+
tags: "${{ fromJson(needs.nf-core-changes.outputs.subworkflows_files) }}"
142132
steps:
143133
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
144134

0 commit comments

Comments
 (0)