-
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: Update for blockchain v5 module. #3149
Conversation
28846e0
to
7348d64
Compare
Are you looking at the SVG being committed or generating using local tooling? I ask because I don't see any text bleedover and I'm wondering if you're using a different layout engine. In fact, the default margin value used by DOT (the layout engine I'm using) is more than 0.01 and as a result, using 0.01 actually makes the generated SVG (with DOT) worse. A value of 0.1 looks good here. Does that work for you? |
This updates the module hierarchy graphviz and module hierarchy diagram to accurately reflect the latest blockchain module dependencies. In particular, blockchain/v5 no longer relies on the gcs, stake, or database modules. It also rearranges the definitions a bit in the graphviz to produce a more balanced diagram.
7348d64
to
cc33b71
Compare
It's really not worth wasting time over, but it looks the same for me even after the margins are changed. I don't know why. It appears setting the margin isn't the problem, it's something to do with our different |
This updates the module hierarchy graphviz and module hierarchy diagram to accurately reflect the latest blockchain module dependencies.
In particular, blockchain/v5 no longer relies on the gcs, stake, or database modules.
It also rearranges the definitions a bit in the graphviz to produce a more balanced diagram.