Skip to content

Conversation

@mark9064
Copy link
Member

Copy link
Contributor

@JF002 JF002 left a comment

Choose a reason for hiding this comment

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

I've just merge InfiniTimeOrg/InfiniTime#2046 so I guess we can merge this one too!

@mark9064
Copy link
Member Author

mark9064 commented May 1, 2024

I still need to update this for the final version of SPI hooks we went with, don't merge just yet!

@mark9064
Copy link
Member Author

mark9064 commented May 2, 2024

Should be ready to go now, builds for me against latest main (when including #142 )

@NeroBurner
Copy link
Collaborator

Could you update this PR to also update the submodule reference to the latest main state of InfiniTime that works with this PR. Afterwards I'll merge right away. Thanks for the fix!

@NeroBurner NeroBurner added the needs-InfiniTime-change PR blocked until InfiniTime upstream change is merged label May 3, 2024
@mark9064
Copy link
Member Author

mark9064 commented May 3, 2024

Rebased on main and updated submodule to InfiniTime main

Copy link
Collaborator

@NeroBurner NeroBurner left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!

@NeroBurner NeroBurner merged commit 1fb70a2 into InfiniTimeOrg:main May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-InfiniTime-change PR blocked until InfiniTime upstream change is merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants