Skip to content

Comments

Update dedent_to to support blocks that are composed of comments#13572

Merged
zanieb merged 2 commits intomainfrom
zb/dedent-comment-block
Oct 1, 2024
Merged

Update dedent_to to support blocks that are composed of comments#13572
zanieb merged 2 commits intomainfrom
zb/dedent-comment-block

Conversation

@zanieb
Copy link
Member

@zanieb zanieb commented Sep 30, 2024

While looking into #13545 I noticed that we return None here if you pass a block of comments. This is annoying because it causes adjust_indentation to fall back to LibCST which panics when it cannot find a statement.

@zanieb zanieb added the internal An internal refactor or improvement label Sep 30, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Sep 30, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

Copy link
Member

@dhruvmanila dhruvmanila left a comment

Choose a reason for hiding this comment

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

Thanks! Can you update the adjust_indent test case at the bottom to include this case?

@dhruvmanila
Copy link
Member

This is annoying because it causes adjust_indentation to fall back to LibCST which panics when it cannot find a statement.

Where does it panic though? I see that the block is being added inside a dummy function body. Does it panic in match_indented_block?

@zanieb zanieb force-pushed the zb/dedent-comment-block branch from ce64e1a to 95a09a1 Compare October 1, 2024 04:33
@zanieb
Copy link
Member Author

zanieb commented Oct 1, 2024

dedent_to returns None and we fail at match_statement — it fails with Failed to create fix for CollapsibleElseIf: Failed to extract statement from source

@zanieb zanieb enabled auto-merge (squash) October 1, 2024 04:36
@zanieb zanieb merged commit 3af3f74 into main Oct 1, 2024
@zanieb zanieb deleted the zb/dedent-comment-block branch October 1, 2024 04:38
@dhruvmanila
Copy link
Member

dedent_to returns None and we fail at match_statement — it fails with Failed to create fix for CollapsibleElseIf: Failed to extract statement from source

Ah right, that's because the following is invalid syntax:

def f():
	# comment
	# comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants