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

Updated MRTK3's minimum supported XRI version XRI 2.3 #11452

Merged
merged 11 commits into from
Apr 6, 2023

Conversation

shaynie
Copy link
Collaborator

@shaynie shaynie commented Apr 6, 2023

Overview

Upgrade XRI to 2.3

Changes

srinjoym
srinjoym previously approved these changes Apr 6, 2023
@srinjoym
Copy link
Contributor

srinjoym commented Apr 6, 2023

The commit diff looks like it contains some history. I think you need to merge the latest microsoft:mrtk3 state into the branch and re-push

AMollis
AMollis previously approved these changes Apr 6, 2023
@shaynie shaynie enabled auto-merge (squash) April 6, 2023 22:42
@shaynie shaynie merged commit 1bf1015 into microsoft:mrtk3 Apr 6, 2023
7 checks passed
@shaynie shaynie deleted the user/shaynie/xri2.3 branch April 6, 2023 22:43
@shaynie shaynie mentioned this pull request Apr 10, 2023
11 tasks
@AMollis AMollis mentioned this pull request Apr 21, 2023
@AMollis AMollis changed the title User/shaynie/xri2.3 Updated MRTK3 to use XRI 2.3 Apr 28, 2023
@AMollis AMollis changed the title Updated MRTK3 to use XRI 2.3 Updated MRTK3's minimum supported XRI version XRI 2.3 Apr 28, 2023
drusk-unity pushed a commit to drusk-unity/MixedRealityToolkit-Unity that referenced this pull request Jun 26, 2023
* Add TransformExtensions class

* Add TransformExtension class and unit tests

* Fix comment new line, change test method names to resemble exiting unit tests

* Add GetChildRecursiveTest

* Move comment to line above code

* Upgrade to XRI 2.3

* Fix sporadically failing unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants