Skip to content
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

linux/[h-k]: add more information link #6227

Merged
merged 7 commits into from
Sep 2, 2021
Merged

Conversation

schneiderl
Copy link
Collaborator

It's been a while since I don't contribute here on tldr.

I'll be trying my best to finish #6062 in the next few days.

In this PR, kde-inhibit was not covered because I did not found a decent source that we can use. Does anyone have suggestions?

@mfrw mfrw added mass changes Changes that affect multiple pages. page edit Changes to an existing page(s). labels Jul 13, 2021
Copy link
Collaborator

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I left a few suggestions below.

pages/linux/hardinfo.md Outdated Show resolved Hide resolved
pages/linux/hwclock.md Outdated Show resolved Hide resolved
pages/linux/imgp.md Outdated Show resolved Hide resolved
@schneiderl
Copy link
Collaborator Author

Do we prefer to use GitHub links whenever they are available? I'll do so on my following PRs then.

Co-authored-by: marchersimon <50295997+marchersimon@users.noreply.github.com>
@marchersimon
Copy link
Collaborator

Do we prefer to use GitHub links whenever they are available? I'll do so on my following PRs then.

We prefer any official source, except it doesn't provide any useful information. Most of the time I just look at the bottom of the manned.org link and see if there's any link to the home page of the command.

@marchersimon
Copy link
Collaborator

This is what I found for kde-inhibit :https://invent.kde.org/plasma/kde-cli-tools/-/tree/master/kdeinhibit.
(Maybe we could also use https://linuxcommandlibrary.com/man/kde-inhibit 😄)

@github-actions
Copy link

Hi all! This thread has not had any recent activity.
Are there any updates? Thanks!

@github-actions github-actions bot added the waiting Issues/PRs with Pending response by the author. label Jul 31, 2021
@marchersimon marchersimon removed the waiting Issues/PRs with Pending response by the author. label Aug 1, 2021
@marchersimon
Copy link
Collaborator

Oh, almost forgot about that this one. @schneiderl, do you want to finish this PR or should we take over?

@schneiderl
Copy link
Collaborator Author

@marchersimon Sorry for the delay. I'm very busy lately. Isn't this ready to be merged?

@marchersimon
Copy link
Collaborator

@schneiderl my comment above about kde-inhibit is still unresolved. But we can just do that one in another PR.

@marchersimon
Copy link
Collaborator

@sbrl, @CleanMachine1 any opinions about the link for kde-inhibit?

Copy link
Member

@blueskyson blueskyson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Collaborator

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's just leave out kde-inhibit for now. Thanks @schneiderl

@marchersimon marchersimon merged commit 65456d0 into main Sep 2, 2021
@marchersimon marchersimon deleted the more-information-linux branch September 2, 2021 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mass changes Changes that affect multiple pages. page edit Changes to an existing page(s).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants