-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
lslocks: add page #3037
lslocks: add page #3037
Conversation
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.
Hi @RH-sdavey, thank you for your contribution! I added some comments for you to review below. I'd like to emphasize the fact that the --pid
and --json
options are not showed in any example. Let me know if there's any problem or if you need any help 🙂
Co-Authored-By: Marco Bonelli <[email protected]>
Co-Authored-By: Marco Bonelli <[email protected]>
Co-Authored-By: Marco Bonelli <[email protected]>
Co-Authored-By: Marco Bonelli <[email protected]>
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.
Are we ready to merge, @mebeim?
pages/linux/lslocks.md
Outdated
|
||
- List locks with defined column headers: | ||
|
||
`lslocks --output PID,COMMAND,PATH` |
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.
Should PID,COMMAND,PATH be within tokens ({{}}
) ?
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.
You're right, it should.
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.
Looks ok to me after @agnivade's comment is taken care of.
Changes made, thanks for merge, will add more updates/new pages later this week. |
Part of #2214
common/
,linux/
, etc.)