Skip to content

Commit

Permalink
Migrate Schema enforcer to use pydanticv2 (#161)
Browse files Browse the repository at this point in the history
* Update pydantic to v2 with necessary changes for tests to pass.
* Remove 3.12 from testing as there are ansible errors.
* Fixed deprecation warnings due to migrating to pydanticv2.
* Add test for load_validators to confirm which ones it loads.
* Updated pydantic env settings and added tests.
* Revert update of poetry.lock. Black tests/test_config_settings.py
* Fixed Flake8 issue.
* Loosened the pydantic version requirements to ^2.0
  • Loading branch information
FragmentedPacket authored Feb 9, 2024
1 parent 613adfc commit d89db1a
Show file tree
Hide file tree
Showing 12 changed files with 224 additions and 145 deletions.
1 change: 1 addition & 0 deletions .bandit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ skips: []
# No need to check for security issues in the test scripts!
exclude_dirs:
- "./tests/"
- "./.venv/"
1 change: 1 addition & 0 deletions .flake8
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,4 @@
# E501: Line length is enforced by Black, so flake8 doesn't need to check it
# W503: Black disagrees with this rule, as does PEP 8; Black wins
ignore = E501, W503
exclude = .venv
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ jobs:
strategy:
fail-fast: true
matrix:
python-version: ["3.8", "3.9", "3.10"]
python-version: ["3.8", "3.9", "3.10", "3.11"]
ansible-core-version: ["2.11.7"]
runs-on: "ubuntu-20.04"
env:
Expand Down Expand Up @@ -159,7 +159,7 @@ jobs:
strategy:
fail-fast: true
matrix:
python-version: ["3.8", "3.9", "3.10"]
python-version: ["3.8", "3.9", "3.10", "3.11"]
ansible-core-version: ["2.11.7"]
runs-on: "ubuntu-20.04"
env:
Expand Down
2 changes: 2 additions & 0 deletions .yamllint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,5 @@ rules:
line-length: "disable"
quoted-strings:
quote-type: "double"
ignore: |
.venv/
Loading

0 comments on commit d89db1a

Please sign in to comment.