Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore CpuId's Debug implementation #159

Merged
merged 4 commits into from May 3, 2023
Merged

Restore CpuId's Debug implementation #159

merged 4 commits into from May 3, 2023

Conversation

khuey
Copy link
Contributor

@khuey khuey commented Apr 27, 2023

This is a regression from 10.7.0. There's no need for CpuIdReader to support Debug, so drop it.

@codecov
Copy link

codecov bot commented Apr 28, 2023

Codecov Report

Merging #159 (9d5d536) into master (927152c) will increase coverage by 1.77%.
The diff coverage is n/a.

❗ Current head 9d5d536 differs from pull request most recent head fe7351c. Consider uploading reports for the commit fe7351c to get more accurate results

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
+ Coverage   36.38%   38.15%   +1.77%     
==========================================
  Files           4        4              
  Lines        4387     4477      +90     
==========================================
+ Hits         1596     1708     +112     
+ Misses       2791     2769      -22     
Impacted Files Coverage Δ
src/lib.rs 46.13% <ø> (+2.87%) ⬆️

... and 1 file with indirect coverage changes

@gz
Copy link
Owner

gz commented May 3, 2023

Thanks for fixing this!

@gz gz merged commit 3d334ea into gz:master May 3, 2023
3 checks passed
@gz
Copy link
Owner

gz commented May 3, 2023

Released as 11.0.1

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.

None yet

2 participants