rust-iot / radio-hal

Embedded rust radio abstraction crate
https://ryan.kurte.nz/notes/2020-01-05-rust-radio
MIT License
69 stars 13 forks source link

Swap to associated trait bounds where viable #13

Closed ryankurte closed 2 years ago

ryankurte commented 2 years ago

relates to #12, any opinions @henrikssn?

henrikssn commented 2 years ago

The change LGTM, is the formatting simply what you get using cargo fmt? In that case, we might want to add a github action that verifies all code is correctly formatted.

I patched this branch into my code base and it all worked fine.

ryankurte commented 2 years ago

The change LGTM, is the formatting simply what you get using cargo fmt? In that case, we might want to add a github action that verifies all code is correctly formatted.

yep, and this is probably a good idea

henrikssn commented 2 years ago

The change LGTM, is the formatting simply what you get using cargo fmt? In that case, we might want to add a github action that verifies all code is correctly formatted.

yep, and this is probably a good idea

Done in #15