Skip to content

Set documentation field in SCIP from doc comment#16559

Merged
bors merged 1 commit into
rust-lang:masterfrom
Wilfred:scip_documentation_field
Feb 15, 2024
Merged

Set documentation field in SCIP from doc comment#16559
bors merged 1 commit into
rust-lang:masterfrom
Wilfred:scip_documentation_field

Conversation

@Wilfred
Copy link
Copy Markdown
Contributor

@Wilfred Wilfred commented Feb 14, 2024

Previously, the documentation field was the same as the text shown to users when they hover over that symbol. The documentation should really just be the doc comment, and as of #16179 the signature is already stored in the signatureDocumentation field.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 14, 2024
Comment thread crates/ide/src/hover.rs Outdated
Comment thread crates/rust-analyzer/src/cli/scip.rs Outdated
@Wilfred Wilfred force-pushed the scip_documentation_field branch from d90cfb8 to b10d3f4 Compare February 14, 2024 17:38
Previously, the documentation field was the same as the text shown to
users when they hover over that symbol. The documentation should
really just be the doc comment, and as of rust-lang#16179 the signature is
already stored in the signatureDocumentation field.
@Wilfred Wilfred force-pushed the scip_documentation_field branch from b10d3f4 to dda641c Compare February 14, 2024 22:46
@lnicola
Copy link
Copy Markdown
Member

lnicola commented Feb 15, 2024

@bors r+

@bors
Copy link
Copy Markdown
Contributor

bors commented Feb 15, 2024

📌 Commit dda641c has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Copy Markdown
Contributor

bors commented Feb 15, 2024

⌛ Testing commit dda641c with merge 8443305...

@bors
Copy link
Copy Markdown
Contributor

bors commented Feb 15, 2024

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing 8443305 to master...

@bors bors merged commit 8443305 into rust-lang:master Feb 15, 2024
@Wilfred Wilfred deleted the scip_documentation_field branch February 15, 2024 22:04
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.

5 participants