Skip to content

Commit 6dde89a

Browse files
author
Phil Varner
committed
rename a few targets, and fix references to them
1 parent b5d59a6 commit 6dde89a

File tree

4 files changed

+40
-24
lines changed

4 files changed

+40
-24
lines changed

.github/pull_request_template.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
**Related Issue(s):** #
1+
**Related Issue(s):**
22

3+
- #
34

45
**Description:**
56

@@ -9,4 +10,4 @@
910
- [ ] Code is formatted and linted (run `pre-commit run --all-files`)
1011
- [ ] Tests pass (run `make test`)
1112
- [ ] Documentation has been updated to reflect changes, if applicable, and docs build successfully (run `make docs`)
12-
- [ ] Changes are added to the [CHANGELOG](https://github.com/stac-utils/pystac/blob/master/CHANGES.md).
13+
- [ ] Changes are added to the [CHANGELOG](https://github.com/stac-utils/pystac/blob/master/CHANGES.md), if applicable
File renamed without changes.

Makefile

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -11,38 +11,35 @@ run_es = docker-compose \
1111
-e APP_PORT=${APP_PORT} \
1212
app-elasticsearch
1313

14-
.PHONY: es-image
15-
es-image:
14+
.PHONY: image
15+
image:
1616
docker-compose build
1717

18-
.PHONY: docker-run-es
19-
docker-run-es: es-image
18+
.PHONY: docker-run
19+
docker-run: image
2020
$(run_es)
2121

22-
.PHONY: docker-shell-es
23-
docker-shell-es:
22+
.PHONY: docker-shell
23+
docker-shell:
2424
$(run_es) /bin/bash
2525

26-
.PHONY: test-es
27-
test-es:
28-
$(run_es) /bin/bash -c 'export && ./scripts/wait-for-it.sh elasticsearch:9200 && cd /app/stac_fastapi/elasticsearch/tests/ && pytest'
26+
.PHONY: test
27+
test:
28+
$(run_es) /bin/bash -c 'export && ./scripts/wait-for-it-es.sh elasticsearch:9200 && cd /app/stac_fastapi/elasticsearch/tests/ && pytest'
2929

30-
.PHONY: run-es-database
31-
run-es-database:
30+
.PHONY: run-database
31+
run-database:
3232
docker-compose run --rm elasticsearch
3333

34-
.PHONY: test
35-
test: test-elasticsearch
36-
3734
.PHONY: pybase-install
3835
pybase-install:
3936
pip install wheel && \
4037
pip install -e ./stac_fastapi/api[dev] && \
4138
pip install -e ./stac_fastapi/types[dev] && \
4239
pip install -e ./stac_fastapi/extensions[dev]
4340

44-
.PHONY: es-install
45-
es-install: pybase-install
41+
.PHONY: install
42+
install: pybase-install
4643
pip install -e ./stac_fastapi/elasticsearch[dev,server]
4744

4845
.PHONY: docs-image

README.md

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,30 @@ Elasticsearch backend for stac-fastapi.
44

55
**WIP** This backend is not yet stable (notice no releases yet), so use the pgstac backend instead.
66

7-
------
8-
#### Running API on localhost:8083
7+
## Development Environment Setup
98

10-
```docker-compose up```
9+
Install [pre-commit](https://pre-commit.com/#install).
1110

12-
------
13-
#### Testing
11+
Prior to commit, run:
1412

15-
```make test-es```
13+
```
14+
pre-commit run --all-files`
15+
```
16+
17+
## Building
18+
19+
```
20+
docker-compose build
21+
```
22+
23+
## Running API on localhost:8083
24+
25+
```
26+
docker-compose up
27+
```
28+
29+
## Testing
30+
31+
```
32+
make test
33+
```

0 commit comments

Comments
 (0)