-
Notifications
You must be signed in to change notification settings - Fork 6
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
Flow improvement #43
base: master
Are you sure you want to change the base?
Flow improvement #43
Conversation
🚀 Deployed on https://preview-43--hedgehog-docs.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clean-up looks good to me, with a few nits here and there.
Co-authored-by: Quentin Monnet <[email protected]>
@@ -12,7 +12,7 @@ After that you will be provided with the credentials to access the software on [ | |||
In order to use the software, log in to the registry using the following command: | |||
|
|||
```bash | |||
docker login ghcr.io | |||
docker login ghcr.io --username provided_user_name --password provided_token_string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not using prompt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am guessing that people will like to use the copy paste. But I am flexible.
```console | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't matter much here - these blocks are console
output, because they contain both commands and their output. However, we won't get the highlight for commands, because the core@control-1
prefix in front of the prompt symbol and commands prevents GitHub to recognise the commands.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I have been looking at the docs as rendered via make serve
I did that mass removal to avoid numbers in the output being colored for reasons I couldn't easily deduce. For examples the first 10 in the ip addresses. The rendering in GitHub is something that I have only started looking at because of this comment. Are you okay with a policy of, no console tag at all since the rendering seems to be spotty be between material-mkdocs and github?
docs/vlab/overview.md
Outdated
docker login ghcr.io --username provided_username --password provided_token | ||
``` | ||
|
||
Hedgehog maintains a utility to install and configure VLAB, called `hhfab`. To install: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hedgehog maintains a utility to install and configure VLAB, called
hhfab
.
I'd move that to the top of the subsection, before mentioning the token required to download this hhfab
you mention :)
Also s/github/GitHub/
No description provided.