Skip to content

Conversation

danparizher
Copy link
Contributor

Summary

Fixes #20198

Copy link
Contributor

github-actions bot commented Sep 2, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

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

Thank you! Just one nit about the test case. Nice use of any_over_expr!

@ntBre ntBre added the bug Something isn't working label Sep 3, 2025
Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

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

Thanks!

@ntBre ntBre changed the title [flake8-comprehensions] Skip C417 when lambda contains yield/yield from (C417) [flake8-comprehensions] Skip C417 when lambda contains yield/yield from Sep 3, 2025
@ntBre ntBre merged commit 200349c into astral-sh:main Sep 3, 2025
35 checks passed
@danparizher danparizher deleted the fix-20198 branch September 3, 2025 21:15
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.

C417 fix introduces a syntax error for yield in lambda expression
2 participants