-
Notifications
You must be signed in to change notification settings - Fork 11
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
bump to embedded-hal
v1.0.0 (embedded-can
)
#6
Comments
Would be greatly appreciated if you could test and PR! One suggestion, I've seen other crates that have embedded-hal 0.x as an optional dependency. Maybe it's also worth doing that here? |
The
This is a bigger job and maybe a separate preliminary PR. |
Hi there! |
embedded-hal
has released v1.0.0 which splits the CAN stuff into its own crateembedded-can
. Same traits looks like.embedded-hal
v0.27.0 Frame: https://docs.rs/embedded-hal/0.2.7/embedded_hal/can/trait.Frame.htmlembedded-can
Frame: https://docs.rs/embedded-can/latest/embedded_can/trait.Frame.htmlAre you open to bumping this if it is indeed a clean swap? I'd happily contribute the change and test on my esp-rs setup w/ 2 MCP2515s.
The text was updated successfully, but these errors were encountered: