mavlink / rust-mavlink

MAVLink library for Rust.
https://mavlink.github.io/rust-mavlink/mavlink/
Apache License 2.0
158 stars 79 forks source link

chore(deps): update 'embedded_hal' #267

Closed danieleades closed 3 months ago

danieleades commented 3 months ago

does CI actually cover this feature?

patrickelectric commented 3 months ago

does CI actually cover this feature?

Yes https://github.com/mavlink/rust-mavlink/actions/runs/10548211116/job/29221931254

danieleades commented 3 months ago

ah i think i see what's going on here. the embedded_hal_02 dependency is essentially a feature which pins to an older version. Therefore this PR should not be merged.

when embedded-io provides feature parity the embedded_hal_02 dependency/feature should be removed, not updated.