Skip to content
This repository was archived by the owner on Mar 20, 2023. It is now read-only.

Commit 856cea4

Browse files
authored
Fix test-as-submodule CI. (#784)
* Also update the GitHub PR template.
1 parent 43e881e commit 856cea4

File tree

2 files changed

+15
-6
lines changed

2 files changed

+15
-6
lines changed

.github/pull_request_template.md

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,15 @@ nrnivmodl mod
2828
- Version: [e.g. master branch]
2929
- Backend: [e.g. CPU]
3030

31-
**Use certain branches for the SimulationStack CI**
32-
33-
CI_BRANCHES:NEURON_BRANCH=master,
31+
**Use certain branches in CI pipelines.**
32+
<!-- You can steer which versions of CoreNEURON dependencies will be used in
33+
the various CI pipelines (GitLab, test-as-submodule) here. Expressions are
34+
of the form PROJ_REF=VALUE, where PROJ is the relevant Spack package name,
35+
transformed to upper case and with hyphens replaced with underscores.
36+
REF may be BRANCH, COMMIT or TAG, with exceptions:
37+
- SPACK_COMMIT and SPACK_TAG are invalid (hpc/gitlab-pipelines limitation)
38+
- NEURON_COMMIT and NEURON_TAG are invalid (test-as-submodule limitation)
39+
These values for NEURON, nmodl and Spack are the defaults and are given
40+
for illustrative purposes; they can safely be removed.
41+
-->
42+
CI_BRANCHES:NEURON_BRANCH=master,NMODL_BRANCH=master,SPACK_BRANCH=develop

.github/workflows/test-as-submodule.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,9 @@ jobs:
5454
env:
5555
GITHUB_PR_BODY: ${{ github.event.pull_request.body }}
5656
run: |
57-
nrn_branch=`echo "${GITHUB_PR_BODY}" | grep "^CI_BRANCHES" \
58-
| awk -v FS="NEURON_BRANCH=" '{print $2}'` \
59-
| awk -v FS="," '{print $1}'
57+
nrn_branch=$(echo "${GITHUB_PR_BODY}" | grep "^CI_BRANCHES" \
58+
| awk -F '[:,]{1}NEURON_BRANCH=' '{print $2}' \
59+
| awk -F ',' '{print $1}')
6060
if [ -z "$nrn_branch" ]; then
6161
nrn_branch=master
6262
fi

0 commit comments

Comments
 (0)