-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Correct the description of the command prompt #8
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
Conversation
Hi @SteveDunn, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! TTYL, MSBOT; |
|
||
a. Add `--layout <path>`, where `<path>` is the location where you want the layout to download to. By default, all languages are downloaded. (See Example A.) | ||
a. Add `--layout <path>`, where `<path>` is the location where you saved the exe in step 1. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution. But this piece isn't quite right, @SteveDunn: the layout path represents the location where the download will be cached. It doesn't have to be the location where you downloaded the initial setup executable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re: the additional text about the editions...we appreciate your feedback, @SteveDunn. We'll take that under advisement and report back with next steps.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, @SteveDunn, we appreciate your input on the example text about all languages being downloaded. We will address that slightly differently than how you've suggested; I'll report back after the work is done. (At that point, we'll most likely close this merge request so that it doesn't conflict with other text changes that are in the queue.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SteveDunn - We're going to opt not to mention editions at this time, and we don't think it's a good idea to mention again that all languages are downloaded again in Step 2b, as that applies directly to Example A and therefore is mentioned in Step 2a. We thank you for submitting this PR, but we're closing it for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment.
Update User Prompt Docs
Visual Studio freshness #8
No description provided.