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

Expand docs section on Visual Studio to mention all three available extensions #14072

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

ericsink
Copy link
Contributor

@ericsink ericsink commented Feb 1, 2023

A recent PR (#14012) by @parthopdas added mention of rust-analyzer.vs, his extension for Visual Studio 2022. I am submitting this PR to request that our extension (SourceGear Rust) be mentioned in that section as well, and also, for completeness, the VS_RustAnalyzer extension, by @cchharris.

Our extension is closed source, so I have clearly disclosed that. For consistency, I included brief mention of the licenses for the other two options as well. Also for the sake of consistency, I added marketplace and GitHub links for all 3.

The previously added paragraph by @parthopdas about his extension has been left intact.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 1, 2023
@lnicola
Copy link
Member

lnicola commented Feb 1, 2023

How did we get from no extensions to three extensions? 😀

Including the license looks a bit weird (we don't do it for the other editor plugins), but I don't feel strongly about this.

@ericsink
Copy link
Contributor Author

ericsink commented Feb 1, 2023

@lnicola

Including the license looks a bit weird

Hmmm. I neglected to review the document to see if any of the other listed plugins are closed source.

That said, my only goal here was to err on the side of appropriate disclosure and fairness.

If the maintainers here decide they want me to remove the license info, I will have no objection.

@lnicola
Copy link
Member

lnicola commented Feb 1, 2023

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 1, 2023

📌 Commit 24d3e93 has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Feb 1, 2023

⌛ Testing commit 24d3e93 with merge a257a58...

@bors
Copy link
Collaborator

bors commented Feb 1, 2023

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing a257a58 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.

4 participants