Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

Commit 8fb6a73

Browse files
Fix bad tox.ini passenv setting (#825) (#826)
Fixes #821. I made a divergence from our testing in qiskit-sphinx-theme to make the INI file more readable, and I didn't properly test it. My bad! This bug meant that we always used `main` as the branch name. I tested this out locally and it works now: ``` GITHUB_REF_NAME="pull" GITHUB_BASE_REF="upstream" tox -e docs ... raise ValueError(GITHUB_BRANCH) ValueError: upstream ```
1 parent 170bd19 commit 8fb6a73

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tox.ini

+2-2
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ setenv =
1010
VIRTUAL_ENV={envdir}
1111
LANGUAGE=en_US
1212
LC_ALL=en_US.utf-8
13-
passenv=
14-
GITHUB_REF_NAME,
13+
passenv =
14+
GITHUB_REF_NAME
1515
GITHUB_BASE_REF
1616
commands =
1717
python -m unittest -v

0 commit comments

Comments
 (0)