Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Don't escape \ and $ in "Extract format expressions" assist #16781

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

DropDemBits
Copy link
Contributor

Fixes #16745

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 7, 2024
@Veykril
Copy link
Member

Veykril commented Mar 7, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Mar 7, 2024

📌 Commit 1f37e5a has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Mar 7, 2024

⌛ Testing commit 1f37e5a with merge 48cb059...

@bors
Copy link
Collaborator

bors commented Mar 7, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 48cb059 to master...

@bors bors merged commit 48cb059 into rust-lang:master Mar 7, 2024
11 checks passed
bors added a commit that referenced this pull request Mar 8, 2024
…ykril

fix: Preserve $ and \ in postfix format completions

`parse_format_exprs` doesn't escape these two as of #16781, so they have to be escaped as a separate step.
@DropDemBits DropDemBits deleted the extract-format-args-escaping branch March 20, 2024 19:36
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.

extract_format_args assist escapes \ character in \n etc.
4 participants