-
-
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
utmpdump: add page #2931
utmpdump: add page #2931
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.
Nice find, @lbonanomi! I've left some comments below for you to review.
Co-Authored-By: lbonanomi <[email protected]>
Co-Authored-By: lbonanomi <[email protected]>
Please thank @Wesalius, I'm just trying to grind-out the checklist in #2214 |
Ah, I see @lbonanomi! Don't feel like you've got to do all the pages there yourself though - and there's no time limit on it :P But thanks so much for helping out :D |
Co-Authored-By: lbonanomi <[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.
Thanks again, @lbonanomi 😺
Oh yeah, and next time if a page is for a list, if you could include something like |
@sbrl: understood. Should I backfill my PRs with issue links? |
This PR is in support of #2214 |
common/
,linux/
, etc.)