Skip to content

Commit

Permalink
Fix pre-commit checks (#247)
Browse files Browse the repository at this point in the history
Address all the feedback that is given when running:
 pre-commit run --all-files
  • Loading branch information
tatiana authored Oct 15, 2024
1 parent 07e5c92 commit c186152
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 4 deletions.
1 change: 1 addition & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ repos:
- id: check-yaml
args:
- --unsafe
exclude: 'tests/fixtures/dag_factory.yml|examples/invalid.yaml|tests/fixtures/invalid_yaml.yml'
- id: debug-statements
- id: end-of-file-fixer
- id: mixed-line-ending
Expand Down
2 changes: 1 addition & 1 deletion tests/fixtures/dag_factory.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,4 +66,4 @@ example_dag4:
tasks:
task_1:
bash_command: !join ['echo ', *arg2]
operator: airflow.operators.bash_operator.BashOperator
operator: airflow.operators.bash_operator.BashOperator
2 changes: 1 addition & 1 deletion tests/fixtures/dag_factory_variables_as_arguments.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,4 @@ example_dag:
dependencies: [task_1]
variables_as_arguments : [
{"variable":"var1","attribute":"bash_command"}
]
]
2 changes: 1 addition & 1 deletion tests/fixtures/invalid_yaml.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ example_dag:
task_3:
operator: airflow.operators.bash_operator.BashOperator
bash_command: 'echo 3'
dependencies: [task_1]
dependencies: [task_1]
2 changes: 1 addition & 1 deletion tests/fixtures/mydocfile.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,4 +70,4 @@ Frugilegas idem progenuit habebat fortissime lateque foci pignora, nec resumit
quam Atrides. Viscera sua Paphon violenta naresque *esse* totas **crimine
resonantia** vulneret ubi lecti omnia. Sua ingens ubi fecit ait est indigestaque
quas haberet da *aerias iaculum nulloque* fluctibus comites cognata, et. Ora
**intrat damna ante** Poemenis annos, et creatis Dianae. Uno lacertis levem?
**intrat damna ante** Poemenis annos, et creatis Dianae. Uno lacertis levem?
1 change: 1 addition & 0 deletions tests/test_dagbuilder.py
Original file line number Diff line number Diff line change
Expand Up @@ -492,6 +492,7 @@ def test_build():
if version.parse(AIRFLOW_VERSION) >= version.parse("1.10.8"):
assert actual["dag"].tags == ["tag1", "tag2"]


def test_get_dag_params_dag_with_task_group():
td = dagbuilder.DagBuilder("test_dag", DAG_CONFIG_TASK_GROUP, DEFAULT_CONFIG)
expected = {
Expand Down

0 comments on commit c186152

Please sign in to comment.