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

Update past embassy-time split #2

Merged
merged 3 commits into from
Aug 19, 2022

Conversation

quentinmit
Copy link
Contributor

@quentinmit quentinmit commented Aug 19, 2022

I'm afraid Embassy just did it again, they split off time into its own embassy-time crate.

@lulf
Copy link
Member

lulf commented Aug 19, 2022

Yeah, although I'm fine to merge this (pending clippy fixes), I think there is at least another crate split coming soon

@quentinmit
Copy link
Contributor Author

Yeah, although I'm fine to merge this (pending clippy fixes), I think there is at least another crate split coming soon

Fixed, I think. I look forward to more refactoring :P

(The treadmill of refactoring is getting pretty painful, but I don't see another option if I want to use new Embassy features in my apps.)

@lulf
Copy link
Member

lulf commented Aug 19, 2022

Yeah, hopefully it will be better when we get some embassy crates on github. Executor, time and the utilities are close I think. Anyway thanks for fixing it 😊

@lulf lulf merged commit 3409729 into drogue-iot:main Aug 19, 2022
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.

None yet

2 participants