-
Notifications
You must be signed in to change notification settings - Fork 317
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
[Feature request] Add a manpage #252
Comments
Hey! I'm ok with adding a manpage, but I personally don't use them and wouldn't know where to start when adding one. I'm more than happy to accept even a half-baked manpage from you, to get us started? |
I'll work on it tomorrow |
There is clap_mangen which supposedly makes it very easy to generate it from the source code. |
This is my draft: manpage.md |
Thank you! If you could create a pull request with that, I'll approve and merge it now. I have a release that's coming out soon as well. I can automate the manpage at a later date, thank you for the research! |
see #253 |
I checked the draft and have an enhancement request for it. Can definitions for the abbreviations in gping's output be added to the manpage? Asking because it would clear up any uncertainties or misunderstandings about what each field in indicating. (Initially I was unsure about what the p95 and t/o fields are indicating, but now after googling I assume p95 to mean 95% of ping times have lower latency than the reported p95 time, and that t/o is the number of pings that have timed out since the graph started.) |
I tried to add clap-mangen in #257, but I'm not really happy with it. I'll try and revisit it. With regards to adding I'm going to close this for now as #253 is merged. thanks for the help @werdahias <3 |
@werdahias is there any movement on the gping Debian package? Anything I can help with? |
Hi,
I'm just waiting for the clapv4 migration to finalize the gping debian package. Since it is a cli program a manpage would make sense. It could life at the root of the project (and installed from there during the build of the package). I can have a stab at it; it would be nice to have one for gping.
The text was updated successfully, but these errors were encountered: