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 the "parse quaternion reading" algorithm #477

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

rakuco
Copy link
Member

@rakuco rakuco commented Jan 25, 2024

Since w3c/deviceorientation#124 and
w3c/orientation-sensor#83, there are no
specifications using this algorithm anymore.


Preview | Diff

Since w3c/deviceorientation#124 and
w3c/orientation-sensor#83, there are no
specifications using this algorithm anymore.
@rakuco rakuco requested a review from reillyeon January 25, 2024 14:29
Copy link
Member

@anssiko anssiko left a comment

Choose a reason for hiding this comment

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

Happy to see this go. 👋

@rakuco
Copy link
Member Author

rakuco commented Jan 25, 2024

Hmm, I seem to have write access to pretty much any other DAS repository but not this one :-D

@reillyeon reillyeon merged commit 1e40156 into w3c:main Jan 25, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Jan 25, 2024
SHA: 1e40156
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@anssiko
Copy link
Member

anssiko commented Jan 25, 2024

@himorin please fix permissions for @rakuco for this repo too.

@himorin
Copy link
Contributor

himorin commented Jan 26, 2024

@rakuco @anssiko fixed. group permission was wrongly set... thank you for poInting this!

@rakuco rakuco deleted the remove-read-quaternions-algorithm branch January 26, 2024 10:28
@rakuco
Copy link
Member Author

rakuco commented Jan 26, 2024

Thanks, @himorin!

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.

4 participants