search

stereolabs / zed-unity

ZED SDK Unity plugin
https://www.stereolabs.com/documentation/guides/using-zed-with-unity/introduction.html
MIT License
243 stars 57 forks source link

BuildScript editor script is not in namespace #281

Closed smitdylan2001 closed 4 months ago

smitdylan2001 commented 5 months ago

Preliminary Checks

Proposal

Put BuildScript.cs (https://github.com/stereolabs/zed-unity/blob/master/ZEDCamera/Assets/ZED/Editor/Scripts/BuildScript.cs) into it's own or in ZED's namespace.

Use-Case

BuildScript is a common script name, so it can give errors that there already is a script called BuildScript in the project. My projects have an internal BuildScript, which conflicted.

Anything else?

There also are more helper scripts which are not in a namespace. It would be nice if they were to keep things organised. This also could enable better Assembly Definition support, improving editor compile times noticibly

github-actions[bot] commented 4 months ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment otherwise it will be automatically closed in 5 days