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

最近の UnitTest #1044

Merged
merged 4 commits into from
Jun 16, 2021
Merged

最近の UnitTest #1044

merged 4 commits into from
Jun 16, 2021

Conversation

ousttrue
Copy link
Contributor

  • [1.0] SpringBone の JsonSchema 変更対応
  • Mtoon マイグレーション。enum 不正値対応
  • ImporterContext.Load 関数の呼び出しでテクスチャが ImporterContext から GameObject に移動することに対応

@ousttrue ousttrue requested a review from Santarh June 16, 2021 09:49
@ousttrue ousttrue changed the title 最近の UniTest 最近の UnitTest Jun 16, 2021
Santarh
Santarh previously approved these changes Jun 16, 2021
Copy link
Contributor

@Santarh Santarh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

よさそう ユーザがさわるクラスのアクセス修飾だけちょっと?

@@ -16,14 +16,14 @@ public class RuntimeGltfInstance : MonoBehaviour, IResponsibilityForDestroyObjec
/// </summary>
public GameObject Root => this.gameObject;

List<(SubAssetKey, UnityEngine.Object)> m_resources = new List<(SubAssetKey, UnityEngine.Object)>();
public List<(SubAssetKey, UnityEngine.Object)> Resources = new List<(SubAssetKey, UnityEngine.Object)>();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

public にするのは IReadOnlyList だけのほうがよくないかな

Copy link
Contributor

@Santarh Santarh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ヨシ!

@ousttrue ousttrue merged commit 9680873 into vrm-c:master Jun 16, 2021
@ousttrue ousttrue deleted the fix/fix_unittest branch October 27, 2021 06:00
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