-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Added usage under Powershell to the manual #2306
Conversation
Added usage under Powershell and Powershell Core and examples of usage under Unix shells, Powershell, and Windows command shell.
This was added to the 1.6 manual. Can you instead add it to the manual at |
@wtlangford Oh, sorry, I missed that file. I'll add it there. Should I also remove it from the 1.6 manual? |
Nah, it's fine to leave it on the 1.6 manual- it'll update on the website the next time I rebuild it, but it won't ever make it to the manpage for 1.6. Having it on the dev manual means it'll be part of the 1.7 release manpage, though |
OK, great. Then I'm done with this pull request, right? Those failed builds are not my fault, I guess. |
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, thank you!
Unix shells: `jq '.["foo"]'` | ||
Powershell: `jq '.[\"foo\"]'` | ||
Windows command shell: `jq ".[\"foo\"]"` |
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.
Is this correct? On my Windows, both PowerShell 5 and PowerShell 7 seem to work the same as Unix Shell....... jq '.["foo"]'
Added usage under Powershell and Powershell Core and examples of usage under Unix shells, Powershell, and Windows command shell into the manual.