-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
Add cURL to the Bash users list in README.md #7948
Conversation
docs/learning-powershell/README.md
Outdated
@@ -100,7 +100,7 @@ Note that all bash commands should continue working on PowerShell session. | |||
| rm -r <folderName> |Remove-Item <folderName> -Recurse |Delete a folder | |||
| find -name build* |Get-ChildItem build* -Recurse |Find a file or folder starting with 'build' | |||
| grep -Rin "sometext" --include="*.cs" |Get-ChildItem -Recurse -Filter *.cs <br> \| Select-String -Pattern "sometext" | Recursively case-insensitive search for text in files | |||
|
|||
| curl dominio.com | Invoke-WebRequest dominio.com | Transfer data from or to a web from differentes protocols |
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.
Maybe Transfer data from web
?
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 think Invoke-RestMethod
is more similar to curl
than Invoke-WebRequest
. Also, the example does work with curl and it's better to use a more generic URL:
curl https://github.com
Invoke-RestMethod https://github.com
For the description, maybe: Transfer data to or from the web
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.
Ok, I'll make the changes.
docs/learning-powershell/README.md
Outdated
@@ -100,7 +100,7 @@ Note that all bash commands should continue working on PowerShell session. | |||
| rm -r <folderName> |Remove-Item <folderName> -Recurse |Delete a folder | |||
| find -name build* |Get-ChildItem build* -Recurse |Find a file or folder starting with 'build' | |||
| grep -Rin "sometext" --include="*.cs" |Get-ChildItem -Recurse -Filter *.cs <br> \| Select-String -Pattern "sometext" | Recursively case-insensitive search for text in files | |||
|
|||
| curl dominio.com | Invoke-WebRequest dominio.com | Transfer data from or to a web from differentes protocols |
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 think Invoke-RestMethod
is more similar to curl
than Invoke-WebRequest
. Also, the example does work with curl and it's better to use a more generic URL:
curl https://github.com
Invoke-RestMethod https://github.com
For the description, maybe: Transfer data to or from the web
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.
LGTM
@vmsilvamolina Thanks for your contribution! |
PR Summary
add the cURL example on the Bash users list
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files 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