Skip to content

Commit 752b41e

Browse files
philvarnergadomski
andauthored
fix: tweaks to docker files and rename compose (#821)
Co-authored-by: Pete Gadomski <[email protected]>
1 parent 12da793 commit 752b41e

File tree

6 files changed

+17
-14
lines changed

6 files changed

+17
-14
lines changed

Diff for: CHANGES.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,7 @@ Full changelog: https://stac-utils.github.io/stac-fastapi/migrations/v3.0.0/#cha
373373
### Added
374374

375375
* Nginx service as second docker-compose stack to demonstrate proxy ([#503](https://github.com/stac-utils/stac-fastapi/pull/503))
376-
* Validation checks in CI using [stac-api-validator](github.com/stac-utils/stac-api-validator) ([#508](https://github.com/stac-utils/stac-fastapi/pull/508))
376+
* Validation checks in CI using [stac-api-validator](https://github.com/stac-utils/stac-api-validator) ([#508](https://github.com/stac-utils/stac-fastapi/pull/508))
377377
* Required links to the sqlalchemy ItemCollection endpoint ([#508](https://github.com/stac-utils/stac-fastapi/pull/508))
378378
* Publication of docker images to GHCR ([#525](https://github.com/stac-utils/stac-fastapi/pull/525))
379379

Diff for: Dockerfile

+6-5
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM python:3.11-slim as base
1+
FROM python:3.12-slim AS base
22

33
# Any python libraries that require system libraries to be installed will likely
44
# need the following packages in order to build
@@ -10,12 +10,13 @@ RUN apt-get update && \
1010

1111
ENV CURL_CA_BUNDLE=/etc/ssl/certs/ca-certificates.crt
1212

13-
FROM base as builder
13+
FROM base AS builder
1414

1515
WORKDIR /app
1616

1717
COPY . /app
1818

19-
RUN python -m pip install -e ./stac_fastapi/types[dev] && \
20-
python -m pip install -e ./stac_fastapi/api[dev] && \
21-
python -m pip install -e ./stac_fastapi/extensions[dev]
19+
RUN python -m pip install \
20+
-e ./stac_fastapi/types[dev] \
21+
-e ./stac_fastapi/api[dev] \
22+
-e ./stac_fastapi/extensions[dev]

Diff for: Dockerfile.docs

+4-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
1-
FROM python:3.11-slim
1+
FROM python:3.12-slim
22

33
# build-essential is required to build a wheel for ciso8601
4-
RUN apt update && apt install -y build-essential
4+
RUN apt update && apt install -y build-essential && \
5+
apt-get clean && \
6+
rm -rf /var/lib/apt/lists/*
57

68
RUN python -m pip install --upgrade pip
79

Diff for: Makefile

+2-2
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,12 @@ install:
1111

1212
.PHONY: docs-image
1313
docs-image:
14-
docker compose -f docker-compose.docs.yml \
14+
docker compose -f compose.docs.yml \
1515
build
1616

1717
.PHONY: docs
1818
docs: docs-image
19-
docker compose -f docker-compose.docs.yml \
19+
docker compose -f compose.docs.yml \
2020
run docs
2121

2222
.PHONY: test

Diff for: README.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,10 @@ Other backends may be available from other sources, search [PyPI](https://pypi.o
7878
Install the packages in editable mode:
7979

8080
```shell
81-
python -m pip install -e \
82-
'stac_fastapi/types[dev]' \
83-
'stac_fastapi/api[dev]' \
84-
'stac_fastapi/extensions[dev]'
81+
python -m pip install \
82+
-e 'stac_fastapi/types[dev]' \
83+
-e 'stac_fastapi/api[dev]' \
84+
-e 'stac_fastapi/extensions[dev]'
8585
```
8686

8787
To run the tests:
File renamed without changes.

0 commit comments

Comments
 (0)