Skip to content

Commit

Permalink
fix: Optionally include dotenv to allow setup
Browse files Browse the repository at this point in the history
The current `setup` target, and any other targets, will fail since the
top-level `include` is a strict `include`. This makes it optional so
that setup/bootstrap succeeds.

Similar to climatepolicyradar/navigator-search-indexer#98.
  • Loading branch information
jesse-c committed Jul 17, 2024
1 parent 58cc1a7 commit 1a43ec7
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
.PHONY: run_local run_docker install test_local build test
include .env
-include .env

setup:
cp .env.example .env

install:
poetry shell
Expand All @@ -9,7 +12,7 @@ run_local:
TARGET_LANGUAGES=en CDN_DOMAIN=cdn.climatepolicyradar.org GOOGLE_APPLICATION_CREDENTIALS=./credentials/google-creds.json python -m cli.run_parser ./data/raw ./data/processed

test_local:
TARGET_LANGUAGES=en CDN_DOMAIN=cdn.climatepolicyradar.org python -m pytest -vvv
TARGET_LANGUAGES=en CDN_DOMAIN=cdn.climatepolicyradar.org poetry run python -m pytest -vvv

build:
docker build -t navigator-document-parser .
Expand Down

0 comments on commit 1a43ec7

Please sign in to comment.