Skip to content

Commit a3e8be9

Browse files
authored
Merge pull request piccolo-orm#121 from piccolo-orm/lint_tests
add linting to test files, and fixed some linting errors
2 parents 4cb2cf4 + db08b9e commit a3e8be9

File tree

4 files changed

+3
-5
lines changed

4 files changed

+3
-5
lines changed

.github/workflows/tests.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ jobs:
2121
pip install -r requirements.txt
2222
pip install -r tests/test-requirements.txt
2323
- name: Mypy
24-
run: mypy piccolo
24+
run: mypy piccolo tests
2525
- name: Flake8
26-
run: flake8 piccolo
26+
run: flake8 piccolo tests
2727

2828
postgres:
2929
runs-on: ubuntu-latest

tests/apps/migrations/auto/test_migration_manager.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
from piccolo.columns.column_types import ForeignKey
33
from unittest.mock import patch, MagicMock
44

5-
from asyncpg.exceptions import UniqueViolationError
5+
from asyncpg.exceptions import UniqueViolationError # type: ignore
66
from piccolo.apps.migrations.auto import MigrationManager
77
from piccolo.apps.migrations.commands.base import BaseMigrationManager
88
from piccolo.columns import Varchar, Text

tests/sqlite_conf.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,3 @@
66

77

88
APP_REGISTRY = AppRegistry(apps=["tests.example_app.piccolo_app"])
9-

tests/table/test_insert.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from unittest.case import TestCase
21
from ..base import DBTestCase
32
from ..example_app.tables import Band, Manager
43

0 commit comments

Comments
 (0)