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

feat: add options for controlling formatting #17

Closed
wants to merge 2 commits into from

Conversation

morgante
Copy link
Contributor

Introduces a custom_help_markdown which can be customized with options, while preserving backwards compatibility.

For my use case, two options were desired:

  • Overriding the title
  • Removing the bottom footer

@janstarke
Copy link

There has been no change to this repository since 2 years, but I was in need for a specific feature. So I forked this repo and incorporated your PR (https://github.com/janstarke/clap-markdown-dfir). I also published the release to https://crates.io. Until now I did not mention you in the Cargo.toml. I invited to as collaborator, so you can add your preferred contact data to the project, if you want to.

Kind regards, Jan

@ConnorGray
Copy link
Owner

@morgante @janstarke My sincerest apologies for the radio silence on merging the PRs that have built up in this repository. I originally wrote this tool at a time I was working on several CLI tools in Rust, but my focus changed and I was working much less actively in Rust.

I'm working through the backlog of PRs and will try to get all of them merged and new version of clap-markdown published soon.

If you have any other issues or PRs, I promise I'll do my best to merge them in a timely manner!

ConnorGray added a commit that referenced this pull request Jun 15, 2024
* feat: add option to override title
* feat: add option to hide footer
* integration: Polish MarkdownOptions type + add test for custom options behavior

---------

Co-authored-by: Morgante Pell <[email protected]>
@ConnorGray
Copy link
Owner

I'm closing this PR as I've now integrated these changes in #25 👷

Thank you again for the contributions, and again my apologies for the delay in getting these integrated!

@ConnorGray ConnorGray closed this Jun 15, 2024
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.

3 participants