Skip to content

Add From impl for pins#364

Merged
bors[bot] merged 1 commit intomasterfrom
unknown repository
Dec 9, 2021
Merged

Add From impl for pins#364
bors[bot] merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Dec 9, 2021

I ran into this when needing to take a generic struct that could contain any gpio pin. Since all the gpio pins have a degrade function I see no reason to not implement From. This allows generic code to work better.

@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 9, 2021

I think the build error is unrelated to my change since that's a dependency resolution issue and my code just adds a From impl.

bors Bot added a commit that referenced this pull request Dec 9, 2021
365: Update defmt crates r=jonas-schievink a=jonas-schievink

Apparently the old version is now causing dependency resolution problems in #364

bors r+

Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.com>
@jonas-schievink
Copy link
Copy Markdown
Contributor

Thanks! The CI issue should have been fixed by #365.

bors r+

@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 9, 2021

Awesome! Would you be able to cut a release after this? I can always pin to git but I've been trying to cut down on the amount of git dependencies in my Cargo.toml files.

@bors
Copy link
Copy Markdown
Contributor

bors Bot commented Dec 9, 2021

Build succeeded:

@bors bors Bot merged commit 169443f into nrf-rs:master Dec 9, 2021
@jonas-schievink
Copy link
Copy Markdown
Contributor

Sadly releases are pretty involved, so I'd like to get some more changes in before I publish one

@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 9, 2021

Understandable! I can pin to git for now. Thanks for merging so fast!

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.

1 participant