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

internal: SCIP: Report the correct version of rust-analyzer in the metadata #15503

Merged
merged 1 commit into from Aug 23, 2023

Conversation

Wilfred
Copy link
Contributor

@Wilfred Wilfred commented Aug 23, 2023

Previously this was hard coded to "0.1". The SCIP protocol allows this to be an arbitrary string:

message ToolInfo {
  // Name of the indexer that produced this index.
  string name = 1;
  // Version of the indexer that produced this index.
  string version = 2;
  // Command-line arguments that were used to invoke this indexer.
  repeated string arguments = 3;
}

so use the same string reported by rust-analyzer --version.

Previously this was hard coded to "0.1". The SCIP protocol allows this
to be an arbitrary string:

```
message ToolInfo {
  // Name of the indexer that produced this index.
  string name = 1;
  // Version of the indexer that produced this index.
  string version = 2;
  // Command-line arguments that were used to invoke this indexer.
  repeated string arguments = 3;
}
```

so use the same string reported by `rust-analyzer --version`.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 23, 2023
@Veykril
Copy link
Member

Veykril commented Aug 23, 2023

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 23, 2023

📌 Commit bc42b99 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Aug 23, 2023

⌛ Testing commit bc42b99 with merge 6444211...

@bors
Copy link
Collaborator

bors commented Aug 23, 2023

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

@bors bors merged commit 6444211 into rust-lang:master Aug 23, 2023
10 checks passed
@lnicola lnicola changed the title SCIP: Report the correct version of rust-analyzer in the metadata internal: SCIP: Report the correct version of rust-analyzer in the metadata Aug 23, 2023
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