-
Notifications
You must be signed in to change notification settings - Fork 296
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
docs: add mermaid diagrams #3705
Conversation
As requested by @themr0c |
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 think yarn.lock changes is missing
You're right. It was ignored by mistake! Added. |
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 think there is something wrong in the yarn.lock changes (like trim being updated, etc)
could you revert yarn.lock changes and then run yarn
command in the root folder and not in the website folder
I think then we should have a better yarn.lock file (most added lines and no replacement lines)
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.
request for changes is only for yarn.lock, other changes are 👍
### What does this PR do? Adds mermaid diagrams to be used within the website. ### Screenshot/screencast of this PR https://github.com/containers/podman-desktop/assets/6422176/d893c7c8-ae28-4e47-90f6-84547b44b944 <!-- Please include a screenshot or a screencast explaining what is doing this PR --> ### What issues does this PR fix or reference? Closes containers#3706 <!-- Include any related issues from Podman Desktop repository (or from another issue tracker). --> N/A ### How to test this PR? Add: ``` ```mermaid graph TD; A-->B; A-->C; B-->D; C-->D; ``` ``` To an example markdown page. Signed-off-by: Charlie Drage <[email protected]>
Updated! Reran I did:
Trim is no longer being updated now. Good catch! |
docs: add mermaid diagrams
What does this PR do?
Adds mermaid diagrams to be used within the website.
Screenshot/screencast of this PR
Screen.Recording.2023-08-29.at.10.01.49.AM.mov
What issues does this PR fix or reference?
Closes #3706
N/A
How to test this PR?
Add: