loomnetwork / unity-sdk

Loom DAppChain SDK for Unity3d
Other
121 stars 34 forks source link

Changed namespaces in all third-party assemblies to have "Loom." prefix #33

Closed ZimM-LostPolygon closed 6 years ago

ZimM-LostPolygon commented 6 years ago

See 3.2 for reasoning behind this PR: https://docs.google.com/document/d/186sKWWotNLUs9tKLFiu0kkfBwTKlnTmbJo9Koi9fSIo/edit#

Having to fork and rebuild all assemblies just to change the namespaces sounded like a lot of tedious work and unneeded support burden. So instead, I whipped up a tool that can search-replace namespaces in assemblies: https://github.com/LostPolygon/AssemblyNamespaceChanger

Also, removed a major part of AsyncAwaitUtil, since it is unused.

ZimM-LostPolygon commented 6 years ago

Don't merge yet, identified some issues with WebGL build.

ZimM-LostPolygon commented 6 years ago

@enlight Build issues are now fixed.