Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add note to contributing guide for developers to have graphviz installed. #665

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

alxmrs
Copy link
Contributor

@alxmrs alxmrs commented Jan 13, 2025

This helped me make sure tests passed in my local environment. I wanted to make this explicit for the next developer (instead of having to grok the actions scripts).

Inlined in order to not violate API boundaries. Trying to put this in a good place ends up leading to a circular import issue.
@tomwhite tomwhite merged commit 08c183d into cubed-dev:main Jan 14, 2025
15 checks passed
@tomwhite
Copy link
Member

Thanks @alxmrs!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants