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

Ensure no console-syntax in reference docs and no md fenced blocks in --help #1874

Merged
merged 5 commits into from
Feb 27, 2023

Conversation

Gedochao
Copy link
Contributor

@Gedochao Gedochao commented Feb 21, 2023

A follow-up for #1872 (comment)
Depends on #1872

Tweaks:

  • console color/font keys (Console.BOLD, Console.RESET etc.) don't go into the reference doc (as they're then not encoded properly)
  • ensure markdown fenced code blocks don't get printed in the command line
  • command line help outputs are still reused in the reference docs, but now console keys get converted into markdown on the fly
  • --full-help detailed messages are included in the reference docs where applicable (HelpMessage.detailedMessage)

@Gedochao Gedochao marked this pull request as ready for review February 22, 2023 08:01
@Gedochao Gedochao added documentation Improvements or additions to documentation help All the issues related to improving help outputs labels Feb 24, 2023
@Gedochao Gedochao changed the title Filter console keys out of reference docs Ensure no console-syntax in reference docs and no md fenced blocks in --help Feb 24, 2023
Copy link
Member

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

LGTM! Just one minor comment

- use `detailedMessage` instead of `message` for commands, where defined
- convert console keys to markdown fence code blocks
- filter out console keys from reference docs
@Gedochao Gedochao merged commit c929892 into VirtusLab:main Feb 27, 2023
@Gedochao Gedochao deleted the adjust-reference-docs branch February 27, 2023 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation help All the issues related to improving help outputs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants