Skip to content

Commit bbc6466

Browse files
committed
ci: fix publish workflow by adding missing dependency
1 parent 8e3b5ac commit bbc6466

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

.github/workflows/publish.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ jobs:
3636
# Upstream issue: https://github.com/docker/docker-py/issues/2807
3737
# Breaking commit: https://github.com/docker/docker-py/commit/c8fba210a222d4f7fde90da8f48db1e7faa637ec
3838
#
39-
pip install --no-cache-dir chartpress==1.0.* six
39+
pip install --no-cache-dir chartpress==1.0.* six pyyaml
4040
4141
- name: Setup push rights to jupyterhub/helm-chart
4242
# This was setup by...

dev-requirements.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ jupyterhub
88
pytest
99
pytest-asyncio
1010
pytest-cov
11+
# pyyaml is used by the scripts in tools/ and could absolutely be replaced by
12+
# using ruamel.yaml. ruamel.yaml is more powerful than pyyaml, and that is
13+
# relevant when writing YAML files back after having loaded them from disk.
14+
pyyaml
1115
requests
1216
ruamel.yaml>=0.15
1317
# FIXME: six is required by docker 5.0.0 but isn't explicitly required any more.

0 commit comments

Comments
 (0)