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

Set documentation field in SCIP from doc comment #16559

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

Wilfred
Copy link
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
crates/ide/src/hover.rs Outdated Show resolved Hide resolved
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.
@lnicola
Copy link
Member

lnicola commented Feb 15, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 15, 2024

📌 Commit dda641c has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Feb 15, 2024

⌛ Testing commit dda641c with merge 8443305...

@bors
Copy link
Collaborator

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
11 checks passed
@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.

None yet

5 participants