Skip to content
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

Package Manager integration/modernization #1

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

Conversation

Koopa1018
Copy link

Added a Package Manifest to the Assets folder, hopefully allowing access from the Package Manager in 2019.3. (Manifest is untested, and the dependency on com.unity.modules.audio probably has a bad version number.)

In order for it to work in 2019.2 even, had to replace all uses of gameObject.renderer and gameObject.audio with equivalent GetComponent<...>() calls, overriding init() whenever needed.

It'll be a pretty clunky package still--none of the samples have been sorted into the proper Samples~ folder--but still. Packagized.

The .audio and .renderer properties on GameObject have been marked obsolete since Rhythmify's creation.
Replaced them with GetComponent<AudioSource>() and GetComponent<Renderer>().
The .audio and .renderer properties on GameObject have been marked obsolete since the repository began.
Replaced them with GetComponent<AudioSource>() and GetComponent<Renderer>().
The .audio and .renderer properties on GameObject have been marked obsolete since the repository began.
Replaced them with GetComponent<AudioSource>() and GetComponent<Renderer>().
Added a package manifest to allow access through the Unity Package Manager.
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.

1 participant