Skip to content

Commit

Permalink
Fix CI and a mypy issue (#9317)
Browse files Browse the repository at this point in the history
  • Loading branch information
Secrus authored Apr 17, 2024
1 parent 5babd38 commit a22bd5d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ tests_task:
# TODO: caching
- alias: mypy
name: "Tests / FreeBSD (Python ${PYTHON_VERSION}) / mypy"
skip: "!changesInclude('.cirrus.yml', 'poetry.lock', 'pyproject.toml', 'src/**.py')"
skip: "!changesInclude('.cirrus.yml', 'poetry.lock', 'pyproject.toml', 'src/**.py', 'tests/**')"
mypy_script: poetry run mypy

status_task:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ jobs:
with:
runner: ${{ matrix.os.image }}
python-version: ${{ matrix.python-version }}
run-mypy: ${{ needs.changes.outputs.src == 'true' }}
run-mypy: ${{ needs.changes.outputs.tests == 'true' }}
run-pytest: ${{ needs.changes.outputs.tests == 'true' }}
run-pytest-export: ${{ needs.changes.outputs.src == 'true' }}
secrets: inherit
Expand Down
4 changes: 2 additions & 2 deletions tests/console/commands/test_add.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,12 @@ def repo_add_default_packages(repo: TestRepository) -> None:
repo.add_package(redis)

cachy010 = get_package("cachy", "0.1.0")
cachy010.extras = {"msgpack": [msgpack_dep]}
cachy010.extras = {canonicalize_name("msgpack"): [msgpack_dep]}
cachy010.add_dependency(msgpack_optional_dep)
repo.add_package(cachy010)

cachy020 = get_package("cachy", "0.2.0")
cachy020.extras = {"msgpack": [get_dependency("msgpack-python")]}
cachy020.extras = {canonicalize_name("msgpack"): [get_dependency("msgpack-python")]}
cachy020.add_dependency(msgpack_dep)
repo.add_package(cachy020)

Expand Down

0 comments on commit a22bd5d

Please sign in to comment.