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

Fix test-as-submodule CI. #784

Merged
merged 3 commits into from
Feb 28, 2022
Merged

Fix test-as-submodule CI. #784

merged 3 commits into from
Feb 28, 2022

Conversation

olupton
Copy link
Contributor

@olupton olupton commented Feb 28, 2022

Description
Previously the closing ` was in the wrong place, meaning that nrn_branch was not set correctly and CoreNEURON PRs were always tested against master of NEURON.

Note that in bash:

$ foo=bar | echo hello
hello
$ echo $foo

$

Also tweak the first awk command so that it would not match CI_BRANCHES:FOONEURON_BRANCH=bar.

This should fix a failure in #772.

Also update the GitHub PR template to include a comment describing usage of the CI_BRANCHES keyword.

Use certain branches for the SimulationStack CI
CI_BRANCHES:NEURON_BRANCH=master

@olupton olupton merged commit 856cea4 into master Feb 28, 2022
@olupton olupton deleted the olupton/submodule-ci-fixup branch February 28, 2022 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants