Skip to content

Commit e590e43

Browse files
author
Marc Coiffier
committed
Séoaration du linter et des jobs de compilation
1 parent 2dca8b1 commit e590e43

2 files changed

Lines changed: 22 additions & 23 deletions

File tree

.github/workflows/idefix-ci-jobs.yml

Lines changed: 6 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -22,21 +22,7 @@ env:
2222
IDEFIX_DIR: ${{ github.workspace }}
2323

2424
jobs:
25-
Linter:
26-
# Don't do this in forks
27-
if: ${{ github.repository == 'idefix-code/idefix' || github.repository == 'glesur/idefix' }}
28-
runs-on: ubuntu-latest
29-
steps:
30-
- uses: actions/checkout@v3
31-
- uses: actions/setup-python@v4
32-
with:
33-
python-version: 3.x
34-
- uses: pre-commit/[email protected]
35-
- uses: pre-commit-ci/[email protected]
36-
if: always()
37-
3825
ShocksHydro:
39-
needs: Linter
4026
runs-on: self-hosted
4127
steps:
4228
- name: Check out repo
@@ -51,7 +37,6 @@ jobs:
5137
run: scripts/ci/run-tests $IDEFIX_DIR/test/HD//MachReflection -all $TESTME_OPTIONS
5238

5339
ParabolicHydro:
54-
needs: Linter
5540
runs-on: self-hosted
5641
steps:
5742
- name: Check out repo
@@ -66,7 +51,6 @@ jobs:
6651
run: scripts/ci/run-tests $IDEFIX_DIR/test/HD/thermalDiffusion -all $TESTME_OPTIONS
6752

6853
ShocksMHD:
69-
needs: Linter
7054
runs-on: self-hosted
7155
steps:
7256
- name: Check out repo
@@ -83,7 +67,6 @@ jobs:
8367
run: scripts/ci/run-tests $IDEFIX_DIR/test/MHD/OrszagTang3D -all $TESTME_OPTIONS
8468

8569
ParabolicMHD:
86-
needs: Linter
8770
runs-on: self-hosted
8871
steps:
8972
- name: Check out repo
@@ -102,7 +85,7 @@ jobs:
10285
run: scripts/ci/run-tests $IDEFIX_DIR/test/MHD/HallWhistler -all $TESTME_OPTIONS
10386

10487
Fargo:
105-
needs: [Linter, ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
88+
needs: [ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
10689
runs-on: self-hosted
10790
steps:
10891
- name: Check out repo
@@ -115,7 +98,7 @@ jobs:
11598
run: scripts/ci/run-tests $IDEFIX_DIR/test/MHD/FargoMHDSpherical -all $TESTME_OPTIONS
11699

117100
ShearingBox:
118-
needs: [Linter, ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
101+
needs: [ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
119102
runs-on: self-hosted
120103
steps:
121104
- name: Check out repo
@@ -128,7 +111,7 @@ jobs:
128111
run: scripts/ci/run-tests $IDEFIX_DIR/test/MHD/ShearingBox -all $TESTME_OPTIONS
129112

130113
SelfGravity:
131-
needs: [Linter, ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
114+
needs: [ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
132115
runs-on: self-hosted
133116
steps:
134117
- name: Check out repo
@@ -147,7 +130,7 @@ jobs:
147130
run: scripts/ci/run-tests $IDEFIX_DIR/test/SelfGravity/DustyCollapse -all $TESTME_OPTIONS
148131

149132
Planet:
150-
needs: [Linter, ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
133+
needs: [ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
151134
runs-on: self-hosted
152135
steps:
153136
- name: Check out repo
@@ -168,7 +151,7 @@ jobs:
168151
run: scripts/ci/run-tests $IDEFIX_DIR/test/Planet/PlanetsIsActiveRK52D -all $TESTME_OPTIONS
169152

170153
Dust:
171-
needs: [Linter, ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
154+
needs: [ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
172155
runs-on: self-hosted
173156
steps:
174157
- name: Check out repo
@@ -181,7 +164,7 @@ jobs:
181164
run: scripts/ci/run-tests $IDEFIX_DIR/test/Dust/DustyWave -all $TESTME_OPTIONS
182165

183166
Braginskii:
184-
needs: [Linter, ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
167+
needs: [ShocksHydro, ParabolicHydro, ShocksMHD, ParabolicMHD]
185168
runs-on: self-hosted
186169
steps:
187170
- name: Check out repo

.github/workflows/idefix-ci.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,21 +10,37 @@ on:
1010
- '.github/ISSUE_TEMPLATE/*'
1111

1212
jobs:
13+
Linter:
14+
# Don't do this in forks
15+
if: ${{ github.repository == 'idefix-code/idefix' || github.repository == 'glesur/idefix' }}
16+
runs-on: ubuntu-latest
17+
steps:
18+
- uses: actions/checkout@v3
19+
- uses: actions/setup-python@v4
20+
with:
21+
python-version: 3.x
22+
- uses: pre-commit/[email protected]
23+
- uses: pre-commit-ci/[email protected]
24+
if: always()
25+
1326
icc-jobs:
27+
needs: Linter
1428
name: CPU Jobs (icc)
1529
uses: ./.github/workflows/idefix-ci-jobs.yml
1630
with:
1731
TESTME_OPTIONS: -Werror
1832
IDEFIX_COMPILER: icc
1933

2034
gcc-jobs:
35+
needs: Linter
2136
name: CPU Jobs (gcc)
2237
uses: ./.github/workflows/idefix-ci-jobs.yml
2338
with:
2439
TESTME_OPTIONS: -Werror
2540
IDEFIX_COMPILER: gcc
2641

2742
cuda-jobs:
43+
needs: Linter
2844
name: CUDA Jobs
2945
uses: ./.github/workflows/idefix-ci-jobs.yml
3046
with:

0 commit comments

Comments
 (0)