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

Unity Package Manager Support #2

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

Conversation

ErnSur
Copy link

@ErnSur ErnSur commented Nov 23, 2024

In response to Splines Issue

Comments

I replaced ResourceUtility calls with Resources API. This was necessary to avoid loading package assets using "Assets" absolute path. Using Resources API in Editor is much simpler and has no optimization drawbacks since assets in Editor/Resources are not included in build

I also had to reformat ResourceUtility before editing it. I advise reviewing the PR commit by commit

See also: Splines PR

The file 'ResourceUtility.cs' contains one or more unusual line terminator characters, like Line Separator (LS) or Paragraph Separator (PS).

It is recommended to remove them from the file. This can be configured via `editor.unusualLineTerminators`.
This was necessary to avoid loading package assets using "Assets" absolute path. Using Resources API in Editor is much simpler and has no optimization drawbacks since assets in Editor/Resources are not included in build
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