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: Only generate snippets for extract_expressions_from_format_string if snippets are supported #17333

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

DropDemBits
Copy link
Contributor

Part of #17332

Fixes extract_expressions_from_format_string so that it doesn't generate snippets if the client doesn't support it.

Used for inserting syntax nodes into existing token trees
- Add doc comments to some `Parse` methods
- Uses `Parse::new` more
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 2, 2024
@Veykril
Copy link
Member

Veykril commented Jun 3, 2024

Thanks!
@bors r+

@bors
Copy link
Collaborator

bors commented Jun 3, 2024

📌 Commit a741bb2 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jun 3, 2024

⌛ Testing commit a741bb2 with merge 5ac4be8...

@bors
Copy link
Collaborator

bors commented Jun 3, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 5ac4be8 to master...

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.

None yet

4 participants