Skip to content

Added Microsoft's open source graph layout engine #9

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hybridherbst
Copy link
Contributor

@Unity-Harry
Copy link
Owner

This would be a wonderful addition, I have some concerns about bundling in the .dll with this package though.
Can you confirm where you got the dll from? I'd like to make sure we're allowed to re-distribute it and confirm the hash :)
Also do you know if this would support all the editor platforms? (Win, Mac, Linux)

@Unity-Harry Unity-Harry self-requested a review August 27, 2020 17:24
@hybridherbst
Copy link
Contributor Author

hybridherbst commented Aug 28, 2020

The DLL comes from Nuget and the code is on GitHub under MIT license. You're right, I forgot to add the license file. Also, it's .NET Standard 2.0 so should work everywhere.

In an ideal world PackMan would already support NuGet out of the box and we could just reference it :)

One idea would be to push the package ecosystem even further, and ask @xoofx for integration in his Unity NuGet registry, I'd like that a lot!

I opened a PR, maybe you can ping Alexandre directly :)
bdovaz/UnityNuGet#15

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

Successfully merging this pull request may close these issues.

2 participants