-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
New function to log object graph #20
Conversation
…attempt at minimizing it
…vior of this constants
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 left a suggestion to reduce avoid some allocations (if manipulating lots of data), and maybe we can optimize a couple more things later.
Another suggestion would be to split building the stack in one method, from generating the mermaid graph in another (maybe).
Yes, this is an issue with the tool, see #21 . |
core review: it is said of a code review from a core team member 🤭 |
Produces beautiful graphs like: