Skip to content

Comments

[flake8-comprehension] Strip parentheses around generators in C400#11607

Merged
charliermarsh merged 1 commit intomainfrom
charlie/c
May 30, 2024
Merged

[flake8-comprehension] Strip parentheses around generators in C400#11607
charliermarsh merged 1 commit intomainfrom
charlie/c

Conversation

@charliermarsh
Copy link
Member

Summary

Closes #11603.

@charliermarsh charliermarsh added bug Something isn't working fixes Related to suggested fixes for violations labels May 30, 2024
@charliermarsh charliermarsh enabled auto-merge (squash) May 30, 2024 03:05
@github-actions
Copy link
Contributor

github-actions bot commented May 30, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@charliermarsh charliermarsh merged commit a8d1328 into main May 30, 2024
@charliermarsh charliermarsh deleted the charlie/c branch May 30, 2024 03:26
MichaReiser pushed a commit that referenced this pull request Jan 17, 2025
…ecessary-generator-set` (`C401`) (#15553)

<!--
Thank you for contributing to Ruff! To help us out with reviewing,
please consider the following:

- Does this pull request include a summary of the change? (See below.)
- Does this pull request include a descriptive title?
- Does this pull request include references to any relevant issues?
-->

## Summary

Fixes parentheses not being stripped in C401. Pretty much the same as
#11607 which fixed it for C400.

## Test Plan
`cargo nextest run`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fixes Related to suggested fixes for violations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

C400 unsafe fix is more unsafe than the documentation suggests

1 participant