Skip to content

Commit

Permalink
Unpin pytest in .pre-commit-config.yaml and remove `type: ignore[attr…
Browse files Browse the repository at this point in the history
…-defined]` comments that are no longer needed as of pytest 7.1.3
  • Loading branch information
Dos Moonen committed Dec 14, 2022
1 parent f4a7906 commit b4082e3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ repos:
additional_dependencies: [
'keyring==23.0.1',
'nox==2021.6.12',
'pytest==7.1.1',
'pytest',
'types-docutils==0.18.3',
'types-setuptools==57.4.14',
'types-freezegun==1.1.9',
Expand Down
4 changes: 2 additions & 2 deletions tests/functional/test_install_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@ def test_do_not_prompt_for_authentication(

@pytest.fixture(params=(True, False), ids=("auth_needed", "auth_not_needed"))
def auth_needed(request: pytest.FixtureRequest) -> bool:
return request.param # type: ignore[attr-defined]
return request.param


@pytest.fixture(
Expand All @@ -386,7 +386,7 @@ def flags(request: pytest.FixtureRequest, auth_needed: bool) -> typing.List[str]
no_input,
force_keyring,
xfail,
) = request.param # type: ignore[attr-defined]
) = request.param

flags = []
if no_input:
Expand Down

0 comments on commit b4082e3

Please sign in to comment.