Skip to content

Commit

Permalink
resolve pylint errors by adjusting import order and adding missing de…
Browse files Browse the repository at this point in the history
…pendencies
  • Loading branch information
ssupecial committed Oct 12, 2024
1 parent 6dfd853 commit 28e6a6c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/airflow_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,21 +13,24 @@ jobs:
with:
python-version: 3.12.7
architecture: x64

- name: Install dependencies
run: pip install apache-airflow pytest

- name: Install Flake8
run: pip install flake8
- name: Run Flake8
run: flake8 airflow/chap9

- name: Install Pylint
run: pip install pylint
- name: Run Pylint
run: find airflow/chap9/ -name "*.py" | xargs pylint --output-format=colorized

- name: Install Black
run: pip install black
- name: Run Black
run: find airflow/chap9/ -name "*.py" | xargs black --check

- name: Install dependencies
run: pip install apache-airflow pytest

- name: Test DAG integrity
run: pytest airflow/chap9/tests/
3 changes: 2 additions & 1 deletion airflow/chap9/dags/bash_command.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import datetime as dt
from airflow import DAG
from airflow.operators.bash import BashOperator
import datetime as dt

with DAG(
dag_id="bash_command",
start_date=dt.datetime(2024, 10, 11),
Expand Down
1 change: 0 additions & 1 deletion airflow/chap9/tests/dags/test_dag_integrity.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import glob
import importlib.util
import os

import pytest
from airflow.models import DAG
from airflow.utils.dag_cycle_tester import check_cycle
Expand Down

0 comments on commit 28e6a6c

Please sign in to comment.