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

Add VRMWatchKit, VRMWatchSceneKit and WatchExample #22

Merged
merged 1 commit into from
Sep 9, 2023

Conversation

naoji3x
Copy link
Contributor

@naoji3x naoji3x commented Sep 9, 2023

@tattn
はじめまして。有用なパッケージの開発ありがとうございます!Apple Watchで使いたい要件が出てきましたので対応版を開発してみました。TargetとしてVRMWatchKit, VRMWatchSceneKitを追加し、サンプルとしてWatchEampleを追加しています。Swiftでの開発は初めてで知見が少なく、ご確認いただけますと幸いです。

@tattn
Copy link
Owner

tattn commented Sep 9, 2023

@naoji3x ありがとうございます!
後ほど実装や動作などを確認させていただきます!

[en]
Thank you for your PR :D
I’ll check the code and the behavior later.

@tattn
Copy link
Owner

tattn commented Sep 9, 2023

@naoji3x
少し確認させていただいたところ、Targetの構成などをより最適化したものにしたいと考えています。
そこで一旦masterブランチではなく、別のブランチにマージし、自分の方で修正しようと思います。

ちなみに、watchOSアプリへの導入にはCarthageを利用していますか?
VRMKitは将来的にCarthageのサポートを終了し、Swift Package Manager (SPM) のみにしようと考えているため、もしSPMのみで問題なければwatchOS対応はSPMのみにしようと思います。

[en]
I checked the PR. I think I wanna optimize the target structure.
So I'll merge it to the other feature branch instead of master branch. And I'll merge it into master branch after fixing it.

I have a question. Do you use Carthage to import VRMKit?
I have a plan to finish Carthage support.
So If you don't use Carthage, I'll support watchOS for Swift Package Manager only.

@naoji3x
Copy link
Contributor Author

naoji3x commented Sep 9, 2023

@tattn
早速のご確認ありがとうございます!Xcode+Swiftは先月から使い始めたばかりで、CarthageもCocoapodsもほとんど使っていません。ですので、SPMのみで全く支障ございません。Target構成の最適化も是非お願いします。

@tattn tattn changed the base branch from main to feature/watchos September 9, 2023 14:45
@tattn
Copy link
Owner

tattn commented Sep 9, 2023

@naoji3x
こちら feature/watchos ブランチにマージします。
master ブランチにマージできる段階になりましたら、バージョンを切りたいと思いますが、それまでは feature/watchos ブランチをご利用ください 🙏

[en]
I'm merging it into feature/watchos branch.
Once the branch is fixed, I'll release it as a new version. Please use the 'feature/watchos' branch for now.

@tattn tattn merged commit 5fededf into tattn:feature/watchos Sep 9, 2023
1 check failed
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