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

[flake8-bugbear] Document explicitly disabling strict zip (B905) #11040

Merged
merged 3 commits into from
Apr 19, 2024
Merged

[flake8-bugbear] Document explicitly disabling strict zip (B905) #11040

merged 3 commits into from
Apr 19, 2024

Conversation

jfrost-mo
Copy link
Contributor

Occasionally you intentionally have iterables of differing lengths. The rule permits this by explicitly adding strict=False, but this was not documented.

Summary

The rule does not currently document how to avoid it when having differing length iterables is intentional. This PR adds that to the rule documentation.

Test Plan

Small documentation change, untested.

Occasionally you intentionally have iterables of differing lengths. The
rule permits this by explicitly adding `strict=False`, but this was not
documented.
Co-authored-by: Vasco Schiavo <[email protected]>
@charliermarsh charliermarsh self-assigned this Apr 19, 2024
@charliermarsh charliermarsh added the documentation Improvements or additions to documentation label Apr 19, 2024
@charliermarsh
Copy link
Member

Thanks!

@charliermarsh charliermarsh enabled auto-merge (squash) April 19, 2024 13:42
@charliermarsh charliermarsh merged commit 7b3c92a into astral-sh:main Apr 19, 2024
17 checks passed
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@jfrost-mo jfrost-mo deleted the B905_clarification branch April 19, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants