Skip to content

Conversation

@eldruin
Copy link
Contributor

@eldruin eldruin commented Jun 18, 2020

Here the adaptions necessary for embedded-hal 1.0.0-alpha.1.

@eldruin eldruin force-pushed the embedded-hal-1.0.0-alpha branch from bae6c23 to 56b09b6 Compare June 19, 2020 12:04
@dbrgn
Copy link
Owner

dbrgn commented Jun 19, 2020

Thanks! I think requiring Rust >=1.34 would be fine, what do you think?

@eldruin
Copy link
Contributor Author

eldruin commented Jun 19, 2020

Good point. The MSRV in embedded-hal had to be raised to Rust 1.35 due to an issue with associated constants in the ADC trait. So far that will be the MSRV of embedded-hal 1.0.0 so actually I would say the MSRV of this crate should be raised to 1.35 as well.
I will revert the commit adding void, update the MSRV and bump the minor version of this crate in this branch.

@dbrgn
Copy link
Owner

dbrgn commented Jun 19, 2020

Makes sense!

@eldruin eldruin force-pushed the embedded-hal-1.0.0-alpha branch from 56b09b6 to 86a3a4e Compare June 19, 2020 16:47
@eldruin
Copy link
Contributor Author

eldruin commented Jun 19, 2020

Done :)

@ryankurte
Copy link
Collaborator

ryankurte commented Jun 19, 2020

Looks good! Blocked on rust-embedded/embedded-hal#177 (which I've been mentioning everywhere useful so we have a list of pending PRs when we ship v1.0.0)

@dbrgn
Copy link
Owner

dbrgn commented Dec 22, 2021

@niclashoyer updated your branch to the latest alpha, so let's continue this discussion in #39.

@dbrgn dbrgn closed this Dec 22, 2021
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.

3 participants