Skip to content

Update to embedded-sensors v0.3.0#21

Merged
jerrysxie merged 1 commit into
OpenDevicePartnership:mainfrom
kurtjd:embedded-sensors
Jul 28, 2025
Merged

Update to embedded-sensors v0.3.0#21
jerrysxie merged 1 commit into
OpenDevicePartnership:mainfrom
kurtjd:embedded-sensors

Conversation

@kurtjd

@kurtjd kurtjd commented Jun 18, 2025

Copy link
Copy Markdown
Member

Updates to embedded-sensors v0.3.0, introducing breaking changes to the API requiring update of tmp108 version from v0.2.0 to v0.3.0.

MSRV changed to 1.84 to be able to use f32::abs from core (which was moved from std). Alternatives are to use silent integer truncation instead of checking for float equality within an epsilon, but that might result in behavior caller does not expect.

tullom
tullom previously approved these changes Jun 19, 2025
jerrysxie
jerrysxie previously approved these changes Jun 19, 2025
@kurtjd kurtjd requested a review from felipebalbi June 19, 2025 20:19
@kurtjd kurtjd dismissed stale reviews from jerrysxie and tullom via 7c905fb June 19, 2025 20:26
@kurtjd kurtjd force-pushed the embedded-sensors branch from 9cc800d to 7c905fb Compare June 19, 2025 20:26
@kurtjd

kurtjd commented Jun 19, 2025

Copy link
Copy Markdown
Member Author

Had to rebase for merge conflicts.

jerrysxie
jerrysxie previously approved these changes Jun 19, 2025
@jerrysxie jerrysxie enabled auto-merge (squash) June 19, 2025 21:35
Comment thread Cargo.toml Outdated

@jerrysxie jerrysxie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kurtjd This seems to be a breaking change. Is this crate on crate.io? Please announce it on Zulip and Discord. Pending that, looking good to me.

@jerrysxie jerrysxie merged commit f2a3374 into OpenDevicePartnership:main Jul 28, 2025
11 checks passed
This was referenced Jul 28, 2025
@kurtjd

kurtjd commented Jul 28, 2025

Copy link
Copy Markdown
Member Author

@kurtjd This seems to be a breaking change. Is this crate on crate.io? Please announce it on Zulip and Discord. Pending that, looking good to me.

Thank you, posted announcement to Discord.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BREAKING CHANGE enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants