Skip to content

Conversation

@fables-tales
Copy link
Owner

No description provided.

@fables-tales fables-tales requested a review from reese December 28, 2025 15:34
@reese
Copy link
Collaborator

reese commented Dec 30, 2025

As far as the approach here, I think probably what you want is basically reverting the changes in 764836d and then changing this bit to have more restrictions -- i.e. return None if there's multiple statements or it's one of whatever list of node types (UnlessNode, IfNode, UntilNode, etc.). (I worry that there will be ~a lot of node types that you'll have to exempt -- and you'll probably want to add a bunch of tests for return/yield/every other keyword where we sometimes don't render them with parens -- hence my general FUD with stripping parens, but c'est la vie.)

(Gonna unassign myself while you're working on this, but feel to reassign when things are working!)

@reese reese removed their request for review December 30, 2025 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants