Skip to content

Commit

Permalink
Docs improvements (#734)
Browse files Browse the repository at this point in the history
* fix table_definition_reader.py

* fix toposort.py

* fix build_export_dag.py

* fix webserver_config_local.py

* fix README.md
  • Loading branch information
nnsW3 authored Jun 26, 2024
1 parent 1f795d8 commit 588269e
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ Check other variables in `dags/ethereumetl_airflow/variables.py`.
### Updating package requirements
Suggested package requirements for Composer are stored in `requirements_airflow.txt`.

You can update the Composer environment using the follow script:
You can update the Composer environment using the following script:
```bash
ENVIRONMENT_NAME="ethereum-etl-0"
LOCAL_REQUIREMENTS_PATH="$(mktemp)"
Expand Down
2 changes: 1 addition & 1 deletion dags/ethereumetl_airflow/build_export_dag.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
extract_field,
)

# When running on Composer, use suggested Data folder for temp storage
# When running on Composer, use the suggested Data folder for temp storage
# This is a folder in the Composer Bucket, mounted locally using gcsfuse
# Overcomes the 10GB ephemerol storage limit on workers (imposed by GKE Autopilot)
# https://cloud.google.com/composer/docs/composer-2/cloud-storage
Expand Down
2 changes: 1 addition & 1 deletion dags/ethereumetl_airflow/parse/table_definition_reader.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ def extract_table_name(file_path):
return file_path.split('/')[-1].replace('.json', '').replace('.sql', '')


# Extracts dependencies and file type from file path and content
# Extracts dependencies and file type from the file path and content
def extract_file_dependencies_and_type(file_path, file_content):
if '.json' in file_path:
filetype = TableDefinitionFileType.JSON
Expand Down
2 changes: 1 addition & 1 deletion dags/ethereumetl_airflow/parse/toposort.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def toposort(data):
and whose values are a set of dependent items. Output is a list of
sets in topological order. The first set consists of items with no
dependences, each subsequent set consists of items that depend upon
items in the preceeding sets."""
items in the preceding sets."""

# Special case empty input.
if len(data) == 0:
Expand Down
2 changes: 1 addition & 1 deletion docker/webserver_config_local.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
# Will allow user self registration
# AUTH_USER_REGISTRATION = True

# The recaptcha it's automatically enabled for user self registration is active and the keys are necessary
# The recaptcha is automatically enabled for user self registration is active and the keys are necessary
# RECAPTCHA_PRIVATE_KEY = PRIVATE_KEY
# RECAPTCHA_PUBLIC_KEY = PUBLIC_KEY

Expand Down

0 comments on commit 588269e

Please sign in to comment.