Skip to content

migrate convert closure to fn to SyntaxEditor#22204

Open
Shourya742 wants to merge 1 commit intorust-lang:masterfrom
Shourya742:2026-04-21-migrate-convert-closure-to-fn
Open

migrate convert closure to fn to SyntaxEditor#22204
Shourya742 wants to merge 1 commit intorust-lang:masterfrom
Shourya742:2026-04-21-migrate-convert-closure-to-fn

Conversation

@Shourya742
Copy link
Copy Markdown
Member

This PR migrates the convert_closure_to_fn assist to SyntaxEditor.

part of #15710 and #18285

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 28, 2026
@Shourya742 Shourya742 force-pushed the 2026-04-21-migrate-convert-closure-to-fn branch from 3087f3d to 43df39b Compare April 28, 2026 11:48
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 28, 2026

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@Shourya742 Shourya742 force-pushed the 2026-04-21-migrate-convert-closure-to-fn branch from 43df39b to 47a2813 Compare April 28, 2026 12:02
@ChayimFriedman2
Copy link
Copy Markdown
Contributor

I see that this PR changes the assist logic considerably. I don't like that and it makes the review harder. Why did you do that?

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 1, 2026

☔ The latest upstream changes (possibly #22131) made this pull request unmergeable. Please resolve the merge conflicts.

@Shourya742
Copy link
Copy Markdown
Member Author

I see that this PR changes the assist logic considerably. I don't like that and it makes the review harder. Why did you do that?

Hmmm... I was trying to replace the range-based edits with the editor. Now, reflecting on your comment, it seems this makes it difficult to review. Let me revert those changes then and make only ted based removal changes...

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants