Skip to content

Conversation

DimitriPapadopoulos
Copy link
Contributor

No description provided.

Copy link
Contributor

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks starting to address this

For the pytest rules we should use tje modern one after the maintainers of flake8-pytest went through the trouble of changing their default after i asked them to

* The linter should not fix by default.
  The --fix option is passed by pre-commit.
@DimitriPapadopoulos DimitriPapadopoulos force-pushed the ruff_0.9.1 branch 2 times, most recently from 382294d to 7c2e0de Compare January 15, 2025 21:02
PYI006 Use `<` or `>=` for `sys.version_info` comparisons
RUF036 `None` not at the end of the type annotation.
RUF043 Pattern passed to `match=` contains metacharacters but is neither escaped nor raw
RUF052 Local dummy variable is accessed
PT001 Use `@pytest.fixture` over `@pytest.fixture()`
PT023 Use `@pytest.mark.skip_commit` over `@pytest.mark.skip_commit()`
@RonnyPfannschmidt RonnyPfannschmidt merged commit 836eb42 into pypa:main Feb 21, 2025
18 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the ruff_0.9.1 branch February 22, 2025 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants