Skip to content

Commit

Permalink
Address @pankajastro's review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
pankajkoti committed Oct 15, 2024
1 parent e8cb297 commit e7beda5
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 12 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,11 @@ jobs:
Static-Check:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ github.event.pull_request.head.sha || github.ref }}

- uses: actions/setup-python@v3
- uses: actions/setup-python@v5
with:
python-version: "3.12"
architecture: "x64"
Expand Down Expand Up @@ -76,11 +76,11 @@ jobs:
- python-version: "3.12"
airflow-version: "2.8"
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: ${{ github.event.pull_request.head.sha || github.ref }}

- uses: actions/cache@v3
- uses: actions/cache@v4
with:
path: |
~/.cache/pip
Expand Down
7 changes: 1 addition & 6 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -45,11 +45,6 @@ dev = [
]
tests = [
"pytest>=6.0",
"pytest-split",
"pytest-dotenv",
"requests-mock",
"pytest-cov",
"pytest-describe",
"pre-commit"
]

Expand Down Expand Up @@ -105,7 +100,7 @@ target-version = ['py39', 'py310', 'py311', 'py312']

[tool.isort]
profile = "black"
known_third_party = ["airflow", "jinja2"]
known_third_party = ["airflow"]

[tool.ruff]
line-length = 120
Expand Down
1 change: 0 additions & 1 deletion scripts/test/unit-cov.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,4 @@ pytest \
--cov-report=term-missing \
--cov-report=xml \
--durations=0 \
-m "not (integration or perf)" \
--ignore=tests/test_example_dags.py
1 change: 0 additions & 1 deletion scripts/test/unit.sh
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
pytest \
-vv \
--durations=0 \
-m "not (integration or perf)" \
--ignore=tests/test_example_dags.py

0 comments on commit e7beda5

Please sign in to comment.