Skip to content

Commit

Permalink
Make Test Now Requires MyPy Compliance
Browse files Browse the repository at this point in the history
  • Loading branch information
derks committed Jul 18, 2024
1 parent c1df8e5 commit f8e9c42
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ Refactoring:
- [PR #681](https://github.com/datafolklabs/cement/pull/681)
- `[dev]` Remove Python 3.5, 3.6, 3.7 Docker Dev Targets
- `[dev]` Added Python 3.13 Dev Target
- `[dev]` Testing now requires typing compliance (`make test` -> `make comply-mypy`)
- `[dev]` Type Annotations (related: [PR #628](https://github.com/datafolklabs/cement/pull/628))
- `[core.arg]` [Issue #692](https://github.com/datafolklabs/cement/issues/692)
- `[core.cache]` [Issue #693](https://github.com/datafolklabs/cement/issues/693)
Expand Down
8 changes: 5 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,15 @@ virtualenv:
@echo "VirtualENV Setup Complete. Now run: eval $(pdm venv activate)"
@echo

comply:
comply: comply-ruff comply-mypy

comply-ruff:
pdm run ruff check cement/ tests/

comply-fix:
comply-ruff-fix:
pdm run ruff check --fix cement/ tests/

comply-typing:
comply-mypy:
pdm run mypy

docs:
Expand Down

0 comments on commit f8e9c42

Please sign in to comment.