Skip to content

Commit

Permalink
fix(docker): Fixed an issue with the Postgres Docker image name (#474)
Browse files Browse the repository at this point in the history
> Why was this change necessary?

When not choosing to add Postgis, the resulting dev Dockerfile for
Postgres ends up prefixed with `n`. This results in a 404 when trying to
download the Docker image.

What happens:
```dockerfile
FROM npostgres:13
```

Expected:
```dockerfile
FROM postgres:13
```

> How does it address the problem?

Removes the template prefix of `cookiecutter.add_postgis` value.

> Are there any side effects?

No
  • Loading branch information
john-fueled authored Apr 18, 2023
1 parent 7295330 commit 2afafb2
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{% if cookiecutter.add_postgis.lower() == "y" %}FROM postgis/postgis:13-3.3{% else %}FROM {{cookiecutter.add_postgis}}postgres:13{% endif %}
{% if cookiecutter.add_postgis.lower() == "y" %}FROM postgis/postgis:13-3.3{% else %}FROM postgres:13{% endif %}

COPY ./compose/dev/postgres/maintenance /usr/local/bin/maintenance
RUN chmod +x /usr/local/bin/maintenance/*
Expand Down

0 comments on commit 2afafb2

Please sign in to comment.