Skip to content

Commit

Permalink
readme and a return value (#46)
Browse files Browse the repository at this point in the history
  • Loading branch information
sanchitram1 authored Jan 2, 2025
1 parent 50a0853 commit 7dffc61
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ Use [Docker](https://docker.com)
### Arguments

Specify these eg. `docker compose -e FOO=bar up`:
Specify these eg. `FOO=bar docker compose up`:

- `FREQUENCY`: Sets how often (in hours) the pipeline should run.
- `TEST`: Runs the loader in test mode when set to true, skipping certain data insertions.
Expand All @@ -40,7 +40,7 @@ Specify these eg. `docker compose -e FOO=bar up`:

> [!NOTE]
> The flag `NO_CACHE` does not mean that files will not get downloaded to your local
> storage (specifically, the ./data directory). It only means that we'll
> storage (specifically, the ./data directory). It only means that we'll
> delete these temporary files from ./data once we're done processing them.
These arguments are all configurable in the `docker-compose.yml` file.
Expand Down
2 changes: 1 addition & 1 deletion core/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,6 @@ def build_query_params(

# env vars could be true or 1, or anything else -- here's a centralized location to
# handle that
def env_vars(env_var: str, default: str):
def env_vars(env_var: str, default: str) -> bool:
var = getenv(env_var, default).lower()
return var == "true" or var == "1"

0 comments on commit 7dffc61

Please sign in to comment.