-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Update README.md #7978
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
Update README.md #7978
Conversation
SteveL-MSFT
left a comment
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.
I don't believe this PR provides much value. The current content is fine.
|
@SteveL-MSFT I actually think that PR provides more value than what is listed in the 'Files changes'. Something we discussed previously. |
|
I think this is the only one that makes more sense:
|
| To make things easy, you can just clone recursively: | ||
|
|
||
| ```sh | ||
| git clone --recursive https://github.com/PowerShell/PowerShell.git |
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.
--recursive is not needed anymore.
Do you want to fix this or merge as is?
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.
I would like this to be removed as --recursive is not needed anymore.
|
@IgoTenz Please fill the PR template above, it helps us create change logs. |
|
@adityapatwardhan I updated the PR description. |
PR Summary
Simple grammatical correction in README.md file.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:to the beginning of the title and remove the prefix when the PR is ready.[feature]if the change is significant or affects feature tests