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 docs for web #122

Merged
merged 4 commits into from
Apr 15, 2021
Merged

Generate docs for web #122

merged 4 commits into from
Apr 15, 2021

Conversation

anaptfox
Copy link
Contributor

@anaptfox anaptfox commented Apr 14, 2021

Description of change

The goal of this PR is to generate a version of the CLI docs that can be easily integrated to the web.

Type of change

  • New feature
  • Bug fix
  • Refactor
  • Documentation

Demo

Before this pull-request

Only docs/cmd were generated. This allows for proper docs in Github.

After this pull-request

We now have both docs/cmd/md and docs/cmd/www.

There are two to not break docs in GitHub, and compile web-specific docs. The differences between the two:

  • use - vs _ (typical for URLs)
  • add frontmatter
  • uses the /cli/command URL (will be how it's represented in the web docs)

@anaptfox anaptfox requested a review from raulb April 14, 2021 15:29
@raulb raulb mentioned this pull request Apr 15, 2021
4 tasks
Copy link
Member

@raulb raulb left a comment

Choose a reason for hiding this comment

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

This is fantastic! Thank you @anaptfox !!!

Instead of making a list of suggestions or things I thought could be nice to have, I decided to open this other pull-request to this branch: #124

Let me know what you think and if you agree, just approve it and I'll make sure we get this shipped.

Thanks again!

@raulb raulb merged commit 3659d01 into master Apr 15, 2021
@raulb raulb deleted the fox-add-frontmatter branch April 15, 2021 15:52
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.

2 participants