Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apply new ruff rules (preview mode) #2235

Merged
merged 4 commits into from
Sep 27, 2024

Conversation

DimitriPapadopoulos
Copy link
Contributor

@DimitriPapadopoulos DimitriPapadopoulos commented Sep 25, 2024

This new rules or issues appear in preview mode (ruff check --preview).

TODO:

  • Add unit tests and/or doctests in docstrings
  • Add docstrings and API docs for any new/modified user-facing classes and functions
  • New/modified features documented in docs/tutorial.rst
  • Changes documented in docs/release.rst
  • GitHub Actions have all passed
  • Test coverage is 100% (Codecov passes)

@DimitriPapadopoulos DimitriPapadopoulos changed the title Apply new ruff fules (preview mode) Apply new ruff rules (preview mode) Sep 25, 2024
@DimitriPapadopoulos DimitriPapadopoulos marked this pull request as draft September 25, 2024 18:33
@jhamman jhamman added this to the 3.0.0 milestone Sep 26, 2024
F841 Local variable is assigned to but never used

This issue is triggered in preview mode.
PYI059 `Generic[]` should always be the last base class

This future rule is triggered in preview mode.
RUF021 Parenthesize `a and b` expressions when chaining `and` and `or`
       together, to make the precedence clear

This future rule is triggered in preview mode.
RUF022 `__all__` is not sorted

This future rule is triggered in preview mode.
@jhamman jhamman merged commit 8e2c660 into zarr-developers:v3 Sep 27, 2024
26 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the ruff_preview branch September 27, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants