-
-
Notifications
You must be signed in to change notification settings - Fork 97
Convert project to Plugins/ folder only #333
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
Convert project to Plugins/ folder only #333
Conversation
… is moved from Assets/Plugins a data generator was used to create EmbeddedAssets. To update any icons should the need arise, one would just simply run the generator again to update it. (see Icon2EmbeddedAssetUtility). Icon2EmbeddedAssetUtility should(tm) find the correct folder automatically when generating icons. All one needs to do is place the icons in the Resources/Icons folder
…realtime-CSG-for-unity into remove-unity-project
…nore meta for readme, etc
…enerating RP materials. instead of using the shader name to generate folders, it now uses URP and HDRP respectively (based on the default shader present in the RP) when creating RP materials, to reduce folder structure depth. additionally, RCSG now uses an assembly definition, which is used to locate RCSG when locating assets.
The inclusion of the package.json is interesting to me. Are you planning on converting Realtime CSG to a package at a later stage? |
This PR does exactly that |
I see, my bad, usually packages don't have a Plugins folder. This one, for example, would normally directly put the API, Editor and Runtime folder at the root. By chance I also happened to notice that the project now also contains an .asmdef, which is a welcome inclusion. However, note that a second .asmdef needs to be created in the Editor folder, as they ignore conventional folder naming rules. Without an Editor .asmdef, it becomes impossible to build, so I urge that this is looked at quickly. This also goes for the folders under API > Foundation and API > Legacy. Refer to Assembly definition and packages for Unity's guidelines. |
The repo is intended to also be used by older versions of unity (back to 5.4), not only newer versions that support package manager, so |
See #334. The mentioned issue has been fixed, as far as I can tell. Please add any additional comments there, thanks. |
This PR:
closes #300
closes #332
It also removes some clutter from the texture browser.
Biggest changes are that the entire repo is no longer a unity project, but just the RealtimeCSG folder. Referencing #332, this should make onboarding a bit smoother, and possibly make updating easier for others who don't make any local changes to RCSG.
Additionally, icons are now embedded in a script (I added a zip with the original icons, and a script generator to update embedded content). This does not change how RealtimeCSG handles icons, but only migrates it to the embedded system. In order to use the script generator, one would simply drop the icons into
Resources/RealtimeCSG/Icons/
anywhere in the project, press "Find Icons", and then press "Convert". A new version ofEmbeddedAssets
will be generated inRealtimeCSG/Plugins/Editor/Scripts/Data/Generated/Icons/
. This path will always point to the RealtimeCSG folder so that it is always generated in the correct location.