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

Remove LICENSE and README meta in .gitignore #105

Merged
merged 1 commit into from
Feb 1, 2021

Conversation

nowsprinting
Copy link
Contributor

変更前

LICENSE.meta と README.md.meta ファイルは存在しますが、.gitignoreの記述によってUniVRMShadersのUPMパッケージからは除外されます。
そのため、UniVRMShadersのUPMをレジストリからインポートしたプロジェクトでは次のエラーが出ます。

Read only asset Packages/com.vrmc.vrmshaders/MToon/LICENSE has not meta file.
Read only asset Packages/com.vrmc.vrmshaders/MToon/README.md has not meta file.

変更後

.gitignoreから.metaの記述を削除することにより、UPMレジストリにpublishするときに LICENSE.meta と README.md.meta ファイルも含まれるようになり、上記エラーは出なくなります。
また、Git URLによるインポートでは従来から LICENSE.meta と README.md.meta ファイルは含まれているため、インポート後の状態に変化はありません。

Before:
LICENSE.meta and README.md.meta file is not included in the UniVRMShaders UPM package registory.
So, “Read only asset Packages/com.vrmc.vrmshaders/MToon/LICENSE has not meta file.” error in imported project.

After:
real .meta files exists, so remove it from .gitignore file will also include the .meta when publish to UPM package registory.
And should not affect imports by git-url.
@Santarh
Copy link
Owner

Santarh commented Feb 1, 2021

LGTM

@Santarh Santarh merged commit cbf459c into Santarh:master Feb 1, 2021
@nowsprinting nowsprinting deleted the remove_ignore_meta branch February 1, 2021 14:55
nowsprinting added a commit to nowsprinting/UniVRM that referenced this pull request Feb 1, 2021
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.

2 participants