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

Don't trigger eq-without-hash when __hash__ is explicitly set to None #6739

Merged
merged 2 commits into from
Aug 21, 2023

Conversation

LaBatata101
Copy link
Contributor

Summary

Closes #6701

Test Plan

cargo test

@github-actions
Copy link
Contributor

github-actions bot commented Aug 21, 2023

PR Check Results

Ecosystem

✅ ecosystem check detected no changes.

Benchmark

Linux

group                                      main                                   pr
-----                                      ----                                   --
formatter/large/dataset.py                 1.00      3.8±0.08ms    10.6 MB/sec    1.03      3.9±0.05ms    10.3 MB/sec
formatter/numpy/ctypeslib.py               1.00   816.7±13.68µs    20.4 MB/sec    1.02   830.7±10.04µs    20.0 MB/sec
formatter/numpy/globals.py                 1.00     88.4±2.47µs    33.4 MB/sec    1.01     89.2±1.13µs    33.1 MB/sec
formatter/pydantic/types.py                1.00  1581.2±29.75µs    16.1 MB/sec    1.02  1617.8±25.86µs    15.8 MB/sec
linter/all-rules/large/dataset.py          1.01     11.8±0.21ms     3.4 MB/sec    1.00     11.8±0.13ms     3.5 MB/sec
linter/all-rules/numpy/ctypeslib.py        1.00      3.2±0.04ms     5.2 MB/sec    1.00      3.2±0.03ms     5.2 MB/sec
linter/all-rules/numpy/globals.py          1.00    445.2±6.68µs     6.6 MB/sec    1.02    453.0±6.17µs     6.5 MB/sec
linter/all-rules/pydantic/types.py         1.00      6.1±0.10ms     4.2 MB/sec    1.01      6.1±0.08ms     4.2 MB/sec
linter/default-rules/large/dataset.py      1.00      6.3±0.10ms     6.5 MB/sec    1.01      6.3±0.06ms     6.4 MB/sec
linter/default-rules/numpy/ctypeslib.py    1.00  1384.5±19.39µs    12.0 MB/sec    1.01  1396.5±18.96µs    11.9 MB/sec
linter/default-rules/numpy/globals.py      1.00    161.6±2.84µs    18.3 MB/sec    1.03    166.9±1.87µs    17.7 MB/sec
linter/default-rules/pydantic/types.py     1.00      2.8±0.04ms     9.0 MB/sec    1.01      2.9±0.04ms     8.9 MB/sec

Windows

group                                      main                                   pr
-----                                      ----                                   --
formatter/large/dataset.py                 1.00      3.8±0.06ms    10.7 MB/sec    1.00      3.8±0.08ms    10.7 MB/sec
formatter/numpy/ctypeslib.py               1.00   775.5±11.87µs    21.5 MB/sec    1.00   777.3±21.25µs    21.4 MB/sec
formatter/numpy/globals.py                 1.00     81.8±1.91µs    36.1 MB/sec    1.00     81.7±1.72µs    36.1 MB/sec
formatter/pydantic/types.py                1.00  1559.0±36.91µs    16.4 MB/sec    1.00  1566.1±32.72µs    16.3 MB/sec
linter/all-rules/large/dataset.py          1.00     13.0±0.22ms     3.1 MB/sec    1.01     13.2±0.25ms     3.1 MB/sec
linter/all-rules/numpy/ctypeslib.py        1.01      3.6±0.08ms     4.7 MB/sec    1.00      3.5±0.07ms     4.7 MB/sec
linter/all-rules/numpy/globals.py          1.00    444.6±7.64µs     6.6 MB/sec    1.01    447.3±9.42µs     6.6 MB/sec
linter/all-rules/pydantic/types.py         1.00      6.8±0.15ms     3.7 MB/sec    1.02      7.0±0.17ms     3.7 MB/sec
linter/default-rules/large/dataset.py      1.00      7.3±0.11ms     5.6 MB/sec    1.00      7.2±0.11ms     5.6 MB/sec
linter/default-rules/numpy/ctypeslib.py    1.01  1534.2±23.75µs    10.9 MB/sec    1.00  1522.9±21.49µs    10.9 MB/sec
linter/default-rules/numpy/globals.py      1.02    178.9±7.60µs    16.5 MB/sec    1.00    174.8±3.96µs    16.9 MB/sec
linter/default-rules/pydantic/types.py     1.01      3.3±0.05ms     7.8 MB/sec    1.00      3.2±0.06ms     7.9 MB/sec

&& matches!(
value.as_ref(),
Expr::Constant(ast::ExprConstant {
value: Constant::None,
Copy link
Member

Choose a reason for hiding this comment

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

Does this need to be None or would assignment work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

While the purpose of PLW1641 is exactly to check for when __eq__ is defined and __hash__ is None (the default when __eq__ is defined but __hash__ is not), I think that if the code explicitly sets __hash__ = None then the lint should not trigger since the programmer explicitly asks for it.

It needs to be None according to the issue description.

Copy link
Member

Choose a reason for hiding this comment

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

👍

@charliermarsh charliermarsh added the bug Something isn't working label Aug 21, 2023
@charliermarsh charliermarsh enabled auto-merge (squash) August 21, 2023 23:43
@charliermarsh charliermarsh merged commit 37f4920 into astral-sh:main Aug 21, 2023
16 checks passed
@charliermarsh
Copy link
Member

Thanks @LaBatata101!

@LaBatata101 LaBatata101 deleted the fix-PLW1641 branch August 25, 2023 20:45
renovate bot added a commit to ixm-one/pytest-cmake-presets that referenced this pull request Aug 25, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ruff](https://beta.ruff.rs/docs)
([source](https://togithub.com/astral-sh/ruff),
[changelog](https://togithub.com/astral-sh/ruff/releases)) | `^0.0.285`
-> `^0.0.286` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>astral-sh/ruff (ruff)</summary>

###
[`v0.0.286`](https://togithub.com/astral-sh/ruff/releases/tag/v0.0.286)

[Compare
Source](https://togithub.com/astral-sh/ruff/compare/v0.0.285...v0.0.286)

<!-- Release notes generated using configuration in .github/release.yml
at v0.0.286 -->

#### What's Changed

##### New Rules

- \[`flake8-bugbear`] Update `function-call-in-argument-default`
(`B008`) to ignore arguments with immutable annotations by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6784
- \[`flake8-bugbear`] Update `mutable-argument-default` (`B006`) to use
`extend-immutable-calls` when determining if annotations are immutable
by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6781
- \[`flake8-pie`] Implement `unnecessary-range-start` (`PIE808`) by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6690
- \[`flake8-pytest-style`] Add autofix for `PT014` by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6698
- \[`pylint`] Implement `no-self-use` (`R6301`) by
[@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6574
- \[`pylint`] Extend `repeated-equality-comparison-target` to check for
mixed orderings and Yoda conditions. by
[@&#8203;tjkuson](https://togithub.com/tjkuson) in
[astral-sh/ruff#6691

##### Settings

- Make isort's `detect-same-package` behavior configurable by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6833

##### Bug Fixes

- Apply RUF017 when start is passed via position by
[@&#8203;hauntsaninja](https://togithub.com/hauntsaninja) in
[astral-sh/ruff#6664
- Use `typing_extensions.TypeAlias` for PYI026 fixes on pre-3.10 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6696
- Ignore multi-comparisons in `repeated-equality-comparison-target` by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6705
- Accept empty inner calls in C414 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6725
- Allow next in FBT exclusions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6729
- Allow `ctypes.WinError()` in flake8-raise by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6731
- Ignore star imports when importing symbols in fixes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6743
- Support C419 autofixes for set comprehensions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6744
- Avoid attempting to fix unconventional submodule imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6745
- Don't trigger `eq-without-hash` when `__hash__` is explicitly set to
`None` by [@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6739
- Avoid E231 if comma is at end-of-line by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6747
- Truncate some messages in diagnostics by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6748
- Fix isolation groups for unused imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6774
- Avoid fixing D200 for docstrings that end in escapes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6779
- Parenthesize expressions prior to LibCST parsing by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6742
- Fallback to end-of-file if ends in trailing continuation by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6789
- Confine repeated-equality-comparison-target to names and attributes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6802
- Fix `native-literals` handling of int literal with attribute access by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6792
- Fix `uncessary-coding-comment` fix when there's leading content by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6775
- Avoid attempting to fix PT018 in multi-statement lines by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6829
- Update ERA100 to apply to commented dictionary items with trailing
comments by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6822
- Avoid parsing other parts of a format specification if replacements
are present by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6858
- Avoid panic in unused arguments rule for parameter-free lambda by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6679
- Avoid `C417` for `lambda` with default and variadic parameters by
[@&#8203;dhruvmanila](https://togithub.com/dhruvmanila) in
[astral-sh/ruff#6752
- Add `networkx` to conventional aliases by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6778
- Skip serializing cell ID if it's None by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6851
- fix is_raw_string for multiple prefixes by
[@&#8203;davidszotten](https://togithub.com/davidszotten) in
[astral-sh/ruff#6865
- Add jupyter notebook cell ids in 4.5+ if missing by
[@&#8203;konstin](https://togithub.com/konstin) in
[astral-sh/ruff#6853

**Full Changelog**:
astral-sh/ruff@v0.0.285...v0.0.286

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ixm-one/pytest-cmake-presets).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jankatins added a commit to jankatins/pr-workflow-example that referenced this pull request Aug 26, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ruff](https://beta.ruff.rs/docs)
([source](https://togithub.com/astral-sh/ruff),
[changelog](https://togithub.com/astral-sh/ruff/releases)) | `0.0.285`
-> `0.0.286` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>astral-sh/ruff (ruff)</summary>

###
[`v0.0.286`](https://togithub.com/astral-sh/ruff/releases/tag/v0.0.286)

[Compare
Source](https://togithub.com/astral-sh/ruff/compare/v0.0.285...v0.0.286)

<!-- Release notes generated using configuration in .github/release.yml
at v0.0.286 -->

#### What's Changed

##### New Rules

- \[`flake8-bugbear`] Update `function-call-in-argument-default`
(`B008`) to ignore arguments with immutable annotations by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6784
- \[`flake8-bugbear`] Update `mutable-argument-default` (`B006`) to use
`extend-immutable-calls` when determining if annotations are immutable
by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6781
- \[`flake8-pie`] Implement `unnecessary-range-start` (`PIE808`) by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6690
- \[`flake8-pytest-style`] Add autofix for `PT014` by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6698
- \[`pylint`] Implement `no-self-use` (`R6301`) by
[@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6574
- \[`pylint`] Extend `repeated-equality-comparison-target` to check for
mixed orderings and Yoda conditions. by
[@&#8203;tjkuson](https://togithub.com/tjkuson) in
[astral-sh/ruff#6691

##### Settings

- Make isort's `detect-same-package` behavior configurable by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6833

##### Bug Fixes

- Apply RUF017 when start is passed via position by
[@&#8203;hauntsaninja](https://togithub.com/hauntsaninja) in
[astral-sh/ruff#6664
- Use `typing_extensions.TypeAlias` for PYI026 fixes on pre-3.10 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6696
- Ignore multi-comparisons in `repeated-equality-comparison-target` by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6705
- Accept empty inner calls in C414 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6725
- Allow next in FBT exclusions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6729
- Allow `ctypes.WinError()` in flake8-raise by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6731
- Ignore star imports when importing symbols in fixes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6743
- Support C419 autofixes for set comprehensions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6744
- Avoid attempting to fix unconventional submodule imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6745
- Don't trigger `eq-without-hash` when `__hash__` is explicitly set to
`None` by [@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6739
- Avoid E231 if comma is at end-of-line by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6747
- Truncate some messages in diagnostics by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6748
- Fix isolation groups for unused imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6774
- Avoid fixing D200 for docstrings that end in escapes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6779
- Parenthesize expressions prior to LibCST parsing by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6742
- Fallback to end-of-file if ends in trailing continuation by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6789
- Confine repeated-equality-comparison-target to names and attributes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6802
- Fix `native-literals` handling of int literal with attribute access by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6792
- Fix `uncessary-coding-comment` fix when there's leading content by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6775
- Avoid attempting to fix PT018 in multi-statement lines by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6829
- Update ERA100 to apply to commented dictionary items with trailing
comments by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6822
- Avoid parsing other parts of a format specification if replacements
are present by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6858
- Avoid panic in unused arguments rule for parameter-free lambda by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6679
- Avoid `C417` for `lambda` with default and variadic parameters by
[@&#8203;dhruvmanila](https://togithub.com/dhruvmanila) in
[astral-sh/ruff#6752
- Add `networkx` to conventional aliases by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6778
- Skip serializing cell ID if it's None by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6851
- fix is_raw_string for multiple prefixes by
[@&#8203;davidszotten](https://togithub.com/davidszotten) in
[astral-sh/ruff#6865
- Add jupyter notebook cell ids in 4.5+ if missing by
[@&#8203;konstin](https://togithub.com/konstin) in
[astral-sh/ruff#6853

**Full Changelog**:
astral-sh/ruff@v0.0.285...v0.0.286

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/jankatins/pr-workflow-example).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi40MC4zIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
renovate bot added a commit to allenporter/flux-local that referenced this pull request Aug 26, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ruff](https://beta.ruff.rs/docs)
([source](https://togithub.com/astral-sh/ruff),
[changelog](https://togithub.com/astral-sh/ruff/releases)) | `==0.0.285`
-> `==0.0.286` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>astral-sh/ruff (ruff)</summary>

###
[`v0.0.286`](https://togithub.com/astral-sh/ruff/releases/tag/v0.0.286)

[Compare
Source](https://togithub.com/astral-sh/ruff/compare/v0.0.285...v0.0.286)

<!-- Release notes generated using configuration in .github/release.yml
at v0.0.286 -->

#### What's Changed

##### New Rules

- \[`flake8-bugbear`] Update `function-call-in-argument-default`
(`B008`) to ignore arguments with immutable annotations by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6784
- \[`flake8-bugbear`] Update `mutable-argument-default` (`B006`) to use
`extend-immutable-calls` when determining if annotations are immutable
by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6781
- \[`flake8-pie`] Implement `unnecessary-range-start` (`PIE808`) by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6690
- \[`flake8-pytest-style`] Add autofix for `PT014` by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6698
- \[`pylint`] Implement `no-self-use` (`R6301`) by
[@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6574
- \[`pylint`] Extend `repeated-equality-comparison-target` to check for
mixed orderings and Yoda conditions. by
[@&#8203;tjkuson](https://togithub.com/tjkuson) in
[astral-sh/ruff#6691

##### Settings

- Make isort's `detect-same-package` behavior configurable by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6833

##### Bug Fixes

- Apply RUF017 when start is passed via position by
[@&#8203;hauntsaninja](https://togithub.com/hauntsaninja) in
[astral-sh/ruff#6664
- Use `typing_extensions.TypeAlias` for PYI026 fixes on pre-3.10 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6696
- Ignore multi-comparisons in `repeated-equality-comparison-target` by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6705
- Accept empty inner calls in C414 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6725
- Allow next in FBT exclusions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6729
- Allow `ctypes.WinError()` in flake8-raise by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6731
- Ignore star imports when importing symbols in fixes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6743
- Support C419 autofixes for set comprehensions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6744
- Avoid attempting to fix unconventional submodule imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6745
- Don't trigger `eq-without-hash` when `__hash__` is explicitly set to
`None` by [@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6739
- Avoid E231 if comma is at end-of-line by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6747
- Truncate some messages in diagnostics by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6748
- Fix isolation groups for unused imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6774
- Avoid fixing D200 for docstrings that end in escapes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6779
- Parenthesize expressions prior to LibCST parsing by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6742
- Fallback to end-of-file if ends in trailing continuation by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6789
- Confine repeated-equality-comparison-target to names and attributes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6802
- Fix `native-literals` handling of int literal with attribute access by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6792
- Fix `uncessary-coding-comment` fix when there's leading content by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6775
- Avoid attempting to fix PT018 in multi-statement lines by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6829
- Update ERA100 to apply to commented dictionary items with trailing
comments by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6822
- Avoid parsing other parts of a format specification if replacements
are present by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6858
- Avoid panic in unused arguments rule for parameter-free lambda by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6679
- Avoid `C417` for `lambda` with default and variadic parameters by
[@&#8203;dhruvmanila](https://togithub.com/dhruvmanila) in
[astral-sh/ruff#6752
- Add `networkx` to conventional aliases by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6778
- Skip serializing cell ID if it's None by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6851
- fix is_raw_string for multiple prefixes by
[@&#8203;davidszotten](https://togithub.com/davidszotten) in
[astral-sh/ruff#6865
- Add jupyter notebook cell ids in 4.5+ if missing by
[@&#8203;konstin](https://togithub.com/konstin) in
[astral-sh/ruff#6853

**Full Changelog**:
astral-sh/ruff@v0.0.285...v0.0.286

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/allenporter/flux-local).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
renovate bot added a commit to allenporter/pyrainbird that referenced this pull request Aug 27, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ruff](https://beta.ruff.rs/docs)
([source](https://togithub.com/astral-sh/ruff),
[changelog](https://togithub.com/astral-sh/ruff/releases)) | `==0.0.285`
-> `==0.0.286` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/ruff/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/ruff/0.0.285/0.0.286?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>astral-sh/ruff (ruff)</summary>

###
[`v0.0.286`](https://togithub.com/astral-sh/ruff/releases/tag/v0.0.286)

[Compare
Source](https://togithub.com/astral-sh/ruff/compare/v0.0.285...v0.0.286)

<!-- Release notes generated using configuration in .github/release.yml
at v0.0.286 -->

#### What's Changed

##### New Rules

- \[`flake8-bugbear`] Update `function-call-in-argument-default`
(`B008`) to ignore arguments with immutable annotations by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6784
- \[`flake8-bugbear`] Update `mutable-argument-default` (`B006`) to use
`extend-immutable-calls` when determining if annotations are immutable
by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6781
- \[`flake8-pie`] Implement `unnecessary-range-start` (`PIE808`) by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6690
- \[`flake8-pytest-style`] Add autofix for `PT014` by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6698
- \[`pylint`] Implement `no-self-use` (`R6301`) by
[@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6574
- \[`pylint`] Extend `repeated-equality-comparison-target` to check for
mixed orderings and Yoda conditions. by
[@&#8203;tjkuson](https://togithub.com/tjkuson) in
[astral-sh/ruff#6691

##### Settings

- Make isort's `detect-same-package` behavior configurable by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6833

##### Bug Fixes

- Apply RUF017 when start is passed via position by
[@&#8203;hauntsaninja](https://togithub.com/hauntsaninja) in
[astral-sh/ruff#6664
- Use `typing_extensions.TypeAlias` for PYI026 fixes on pre-3.10 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6696
- Ignore multi-comparisons in `repeated-equality-comparison-target` by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6705
- Accept empty inner calls in C414 by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6725
- Allow next in FBT exclusions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6729
- Allow `ctypes.WinError()` in flake8-raise by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6731
- Ignore star imports when importing symbols in fixes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6743
- Support C419 autofixes for set comprehensions by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6744
- Avoid attempting to fix unconventional submodule imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6745
- Don't trigger `eq-without-hash` when `__hash__` is explicitly set to
`None` by [@&#8203;LaBatata101](https://togithub.com/LaBatata101) in
[astral-sh/ruff#6739
- Avoid E231 if comma is at end-of-line by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6747
- Truncate some messages in diagnostics by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6748
- Fix isolation groups for unused imports by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6774
- Avoid fixing D200 for docstrings that end in escapes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6779
- Parenthesize expressions prior to LibCST parsing by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6742
- Fallback to end-of-file if ends in trailing continuation by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6789
- Confine repeated-equality-comparison-target to names and attributes by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6802
- Fix `native-literals` handling of int literal with attribute access by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6792
- Fix `uncessary-coding-comment` fix when there's leading content by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6775
- Avoid attempting to fix PT018 in multi-statement lines by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6829
- Update ERA100 to apply to commented dictionary items with trailing
comments by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6822
- Avoid parsing other parts of a format specification if replacements
are present by [@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6858
- Avoid panic in unused arguments rule for parameter-free lambda by
[@&#8203;charliermarsh](https://togithub.com/charliermarsh) in
[astral-sh/ruff#6679
- Avoid `C417` for `lambda` with default and variadic parameters by
[@&#8203;dhruvmanila](https://togithub.com/dhruvmanila) in
[astral-sh/ruff#6752
- Add `networkx` to conventional aliases by
[@&#8203;zanieb](https://togithub.com/zanieb) in
[astral-sh/ruff#6778
- Skip serializing cell ID if it's None by
[@&#8203;harupy](https://togithub.com/harupy) in
[astral-sh/ruff#6851
- fix is_raw_string for multiple prefixes by
[@&#8203;davidszotten](https://togithub.com/davidszotten) in
[astral-sh/ruff#6865
- Add jupyter notebook cell ids in 4.5+ if missing by
[@&#8203;konstin](https://togithub.com/konstin) in
[astral-sh/ruff#6853

**Full Changelog**:
astral-sh/ruff@v0.0.285...v0.0.286

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/allenporter/pyrainbird).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PLW1641/eq-without-hash should not trigger when explicitly setting __hash__ = None
4 participants