Skip to content

Commit 04d05ad

Browse files
committed
test: add comment that ci tests don't need pytest-env.
redis run in ci doesn't have passwords.
1 parent 22653b5 commit 04d05ad

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

.github/workflows/ci-test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,8 @@ jobs:
116116
- name: Update pip
117117
run: python -m pip install --upgrade pip
118118

119+
# note pytest-env is not needed for redis password as there is
120+
# no password on CI's redis.
119121
- name: Install pytest and other packages needed for running tests
120122
run: pip install flake8 mock pytest pytest-cov requests sphinx-tabs
121123

.travis.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,8 @@ install:
132132
- pip install redis
133133
- pip install pytz whoosh pyjwt requests
134134
- pip install jinja2
135+
# note pytest-env is not needed for redis password as there is
136+
# no password on CI's redis.
135137
- pip install pytest-cov
136138
- pip install beautifulsoup4
137139
- if [[ $TRAVIS_PYTHON_VERSION != "3.4"* ]]; then pip install docutils; fi

0 commit comments

Comments
 (0)