Skip to content

Commit 646758e

Browse files
authored
ci: Checkout submodules (#106)
This is necessary for API repositories and it is harmless for the rest (and usually if any other repository have submodules, we'll want to pull them), so we include it for all.
2 parents 286a484 + ff97220 commit 646758e

File tree

8 files changed

+57
-1
lines changed
  • .github/workflows
  • cookiecutter/{{cookiecutter.github_repo_name}}/.github/workflows
  • tests_golden/integration/test_cookiecutter_generation
    • actor/frequenz-actor-test/.github/workflows
    • api/frequenz-api-test/.github/workflows
    • app/frequenz-app-test/.github/workflows
    • lib/frequenz-test-python/.github/workflows
    • model/frequenz-model-test/.github/workflows

8 files changed

+57
-1
lines changed

.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ jobs:
3737
steps:
3838
- name: Fetch sources
3939
uses: actions/checkout@v3
40+
with:
41+
submodules: true
4042

4143
- name: Set up Python
4244
uses: actions/setup-python@v4
@@ -61,6 +63,8 @@ jobs:
6163
steps:
6264
- name: Fetch sources
6365
uses: actions/checkout@v3
66+
with:
67+
submodules: true
6468

6569
- name: Set up Python
6670
uses: actions/setup-python@v4
@@ -90,6 +94,8 @@ jobs:
9094
steps:
9195
- name: Fetch sources
9296
uses: actions/checkout@v3
97+
with:
98+
submodules: true
9399

94100
- name: Setup Git user and e-mail
95101
uses: frequenz-floss/setup-git-user@v2
@@ -167,6 +173,8 @@ jobs:
167173
- name: Fetch sources
168174
if: steps.mike-metadata.outputs.version
169175
uses: actions/checkout@v3
176+
with:
177+
submodules: true
170178

171179
- name: Setup Git user and e-mail
172180
if: steps.mike-metadata.outputs.version

RELEASE_NOTES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,4 @@
1414

1515
## Bug Fixes
1616

17-
<!-- Here goes notable bug fixes that are worth a special mention or explanation -->
17+
- Cookiecutter: Now the CI workflow will checkout the submodules.

cookiecutter/{{cookiecutter.github_repo_name}}/.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,8 @@ jobs:
6262
steps:
6363
- name: Fetch sources
6464
uses: actions/checkout@v3
65+
with:
66+
submodules: true
6567

6668
- name: Set up Python
6769
uses: actions/setup-python@v4
@@ -86,6 +88,8 @@ jobs:
8688
steps:
8789
- name: Fetch sources
8890
uses: actions/checkout@v3
91+
with:
92+
submodules: true
8993

9094
- name: Set up Python
9195
uses: actions/setup-python@v4
@@ -115,6 +119,8 @@ jobs:
115119
steps:
116120
- name: Fetch sources
117121
uses: actions/checkout@v3
122+
with:
123+
submodules: true
118124

119125
- name: Setup Git user and e-mail
120126
uses: frequenz-floss/setup-git-user@v2
@@ -192,6 +198,8 @@ jobs:
192198
- name: Fetch sources
193199
if: steps.mike-metadata.outputs.version
194200
uses: actions/checkout@v3
201+
with:
202+
submodules: true
195203

196204
- name: Setup Git user and e-mail
197205
if: steps.mike-metadata.outputs.version

tests_golden/integration/test_cookiecutter_generation/actor/frequenz-actor-test/.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ jobs:
3737
steps:
3838
- name: Fetch sources
3939
uses: actions/checkout@v3
40+
with:
41+
submodules: true
4042

4143
- name: Set up Python
4244
uses: actions/setup-python@v4
@@ -61,6 +63,8 @@ jobs:
6163
steps:
6264
- name: Fetch sources
6365
uses: actions/checkout@v3
66+
with:
67+
submodules: true
6468

6569
- name: Set up Python
6670
uses: actions/setup-python@v4
@@ -90,6 +94,8 @@ jobs:
9094
steps:
9195
- name: Fetch sources
9296
uses: actions/checkout@v3
97+
with:
98+
submodules: true
9399

94100
- name: Setup Git user and e-mail
95101
uses: frequenz-floss/setup-git-user@v2
@@ -167,6 +173,8 @@ jobs:
167173
- name: Fetch sources
168174
if: steps.mike-metadata.outputs.version
169175
uses: actions/checkout@v3
176+
with:
177+
submodules: true
170178

171179
- name: Setup Git user and e-mail
172180
if: steps.mike-metadata.outputs.version

tests_golden/integration/test_cookiecutter_generation/api/frequenz-api-test/.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,8 @@ jobs:
6060
steps:
6161
- name: Fetch sources
6262
uses: actions/checkout@v3
63+
with:
64+
submodules: true
6365

6466
- name: Set up Python
6567
uses: actions/setup-python@v4
@@ -84,6 +86,8 @@ jobs:
8486
steps:
8587
- name: Fetch sources
8688
uses: actions/checkout@v3
89+
with:
90+
submodules: true
8791

8892
- name: Set up Python
8993
uses: actions/setup-python@v4
@@ -113,6 +117,8 @@ jobs:
113117
steps:
114118
- name: Fetch sources
115119
uses: actions/checkout@v3
120+
with:
121+
submodules: true
116122

117123
- name: Setup Git user and e-mail
118124
uses: frequenz-floss/setup-git-user@v2
@@ -190,6 +196,8 @@ jobs:
190196
- name: Fetch sources
191197
if: steps.mike-metadata.outputs.version
192198
uses: actions/checkout@v3
199+
with:
200+
submodules: true
193201

194202
- name: Setup Git user and e-mail
195203
if: steps.mike-metadata.outputs.version

tests_golden/integration/test_cookiecutter_generation/app/frequenz-app-test/.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ jobs:
3737
steps:
3838
- name: Fetch sources
3939
uses: actions/checkout@v3
40+
with:
41+
submodules: true
4042

4143
- name: Set up Python
4244
uses: actions/setup-python@v4
@@ -61,6 +63,8 @@ jobs:
6163
steps:
6264
- name: Fetch sources
6365
uses: actions/checkout@v3
66+
with:
67+
submodules: true
6468

6569
- name: Set up Python
6670
uses: actions/setup-python@v4
@@ -90,6 +94,8 @@ jobs:
9094
steps:
9195
- name: Fetch sources
9296
uses: actions/checkout@v3
97+
with:
98+
submodules: true
9399

94100
- name: Setup Git user and e-mail
95101
uses: frequenz-floss/setup-git-user@v2
@@ -167,6 +173,8 @@ jobs:
167173
- name: Fetch sources
168174
if: steps.mike-metadata.outputs.version
169175
uses: actions/checkout@v3
176+
with:
177+
submodules: true
170178

171179
- name: Setup Git user and e-mail
172180
if: steps.mike-metadata.outputs.version

tests_golden/integration/test_cookiecutter_generation/lib/frequenz-test-python/.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ jobs:
3737
steps:
3838
- name: Fetch sources
3939
uses: actions/checkout@v3
40+
with:
41+
submodules: true
4042

4143
- name: Set up Python
4244
uses: actions/setup-python@v4
@@ -61,6 +63,8 @@ jobs:
6163
steps:
6264
- name: Fetch sources
6365
uses: actions/checkout@v3
66+
with:
67+
submodules: true
6468

6569
- name: Set up Python
6670
uses: actions/setup-python@v4
@@ -90,6 +94,8 @@ jobs:
9094
steps:
9195
- name: Fetch sources
9296
uses: actions/checkout@v3
97+
with:
98+
submodules: true
9399

94100
- name: Setup Git user and e-mail
95101
uses: frequenz-floss/setup-git-user@v2
@@ -167,6 +173,8 @@ jobs:
167173
- name: Fetch sources
168174
if: steps.mike-metadata.outputs.version
169175
uses: actions/checkout@v3
176+
with:
177+
submodules: true
170178

171179
- name: Setup Git user and e-mail
172180
if: steps.mike-metadata.outputs.version

tests_golden/integration/test_cookiecutter_generation/model/frequenz-model-test/.github/workflows/ci.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ jobs:
3737
steps:
3838
- name: Fetch sources
3939
uses: actions/checkout@v3
40+
with:
41+
submodules: true
4042

4143
- name: Set up Python
4244
uses: actions/setup-python@v4
@@ -61,6 +63,8 @@ jobs:
6163
steps:
6264
- name: Fetch sources
6365
uses: actions/checkout@v3
66+
with:
67+
submodules: true
6468

6569
- name: Set up Python
6670
uses: actions/setup-python@v4
@@ -90,6 +94,8 @@ jobs:
9094
steps:
9195
- name: Fetch sources
9296
uses: actions/checkout@v3
97+
with:
98+
submodules: true
9399

94100
- name: Setup Git user and e-mail
95101
uses: frequenz-floss/setup-git-user@v2
@@ -167,6 +173,8 @@ jobs:
167173
- name: Fetch sources
168174
if: steps.mike-metadata.outputs.version
169175
uses: actions/checkout@v3
176+
with:
177+
submodules: true
170178

171179
- name: Setup Git user and e-mail
172180
if: steps.mike-metadata.outputs.version

0 commit comments

Comments
 (0)