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

Generate manpages #375

Merged
merged 3 commits into from
Nov 20, 2023
Merged

Conversation

guilherme-puida
Copy link
Contributor

@guilherme-puida guilherme-puida commented Nov 20, 2023

This PR adds a new subcommand (man) that generates manpages for all dasel subcommands (including man itself). Closes #201.

This is the first implementation that came to mind, so it may not be exactly what you had in mind. Let me know if you want me to change anything.

I also wasn't sure if adding tests for this subcommand was worth the trouble. Let me know if you want me to implement them anyway.

Thanks!

This commit adds a new subcommand (man) that is used to generate the man
tree for all dasel subcommands.
@guilherme-puida
Copy link
Contributor Author

I went ahead and implemented tests for the man subcommand as well. It was not as hard as I thought :^)

@TomWright
Copy link
Owner

Thanks for adding the test. I'll have a quick review of this tonight and hopefully get it merged 👍

@TomWright TomWright self-requested a review November 20, 2023 17:01
@TomWright TomWright merged commit 79476ca into TomWright:master Nov 20, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include a man page
2 participants