Skip to content

Commit cefd623

Browse files
authored
Fix trailing blank line in dev only Dockerfile (#196)
2 parents de15bec + d0b0d70 commit cefd623

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

template/Dockerfile.jinja

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
1010

1111
# Set up a virtual environment and put it in PATH
1212
RUN python -m venv /venv
13-
ENV PATH=/venv/bin:$PATH
14-
{% if docker %}
13+
ENV PATH=/venv/bin:$PATH{% if docker %}
14+
1515
# The build stage installs the context into the venv
1616
FROM developer as build
1717
COPY . /context

tests/test_example.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,10 +114,12 @@ def test_template_no_docs(tmp_path: Path):
114114
run("./venv/bin/tox -p")
115115

116116

117-
def test_template_no_docker_has_no_docs(tmp_path: Path):
117+
def test_template_no_docker_has_no_docs_and_works(tmp_path: Path):
118118
copy_project(tmp_path, docker=False)
119119
container_doc = tmp_path / "docs" / "how-to" / "run-container.md"
120120
assert not container_doc.exists()
121+
run = make_venv(tmp_path)
122+
run("./venv/bin/tox -p")
121123

122124

123125
def test_bad_repo_name(tmp_path: Path):

0 commit comments

Comments
 (0)