-
Notifications
You must be signed in to change notification settings - Fork 166
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
doc: add architecture to release overview #3996
Conversation
Add an architecture diagram to the release overview. Keep the existing flow chart as an "In Detail" section.
6cf13c2
to
7655d35
Compare
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.
LGTM
The preview doesn't work for me. I see infinite loading spinners. |
@targos strange I just checked again and it was ok for me. Which browser are you using? |
Firefox |
Firefox on Windows seemed to display for me as well so not sure what's going on. |
Ok, I reloaded the page a few times and it worked. |
FWIW if relying on GitHub to render the mermaid becomes an issue, we can use https://github.com/mermaid-js/mermaid-cli to render the mermaid definition as SVG. That would introduce a secondary build step to update the diagram, so I'd like to avoid that if possible. |
Add an architecture diagram to the release overview. Keep the existing flow chart as an "In Detail" section.
Preview: https://github.com/richardlau/build/blob/release-overview/doc/release-overview.md