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

Add CpuSvnVerifier #38

Merged
merged 1 commit into from
Mar 14, 2023
Merged

Add CpuSvnVerifier #38

merged 1 commit into from
Mar 14, 2023

Conversation

nick-mobilecoin
Copy link
Collaborator

@nick-mobilecoin nick-mobilecoin commented Mar 9, 2023

Add CpuSvnVerifier for verifying the CpuSvn in a ReportBody meets the minimum required CpuSvn

@nick-mobilecoin nick-mobilecoin requested a review from jcape as a code owner March 9, 2023 00:14
@nick-mobilecoin nick-mobilecoin requested review from a team and samdealy and removed request for a team March 9, 2023 00:17
@codecov
Copy link

codecov bot commented Mar 9, 2023

Codecov Report

Merging #38 (1c4a61b) into nick/report-data-mask (35f4881) will increase coverage by 0.54%.
The diff coverage is 100.00%.

@@                    Coverage Diff                    @@
##           nick/report-data-mask      #38      +/-   ##
=========================================================
+ Coverage                  94.84%   95.39%   +0.54%     
=========================================================
  Files                          2        2              
  Lines                        660      738      +78     
=========================================================
+ Hits                         626      704      +78     
  Misses                        34       34              
Impacted Files Coverage Δ
verifier/src/lib.rs 90.51% <100.00%> (ø)
verifier/src/report_body.rs 99.27% <100.00%> (+0.17%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@samdealy
Copy link
Contributor

Please add a PR description before merging.

@nick-mobilecoin
Copy link
Collaborator Author

@nick-mobilecoin queued this pull request to merge with Graphite.

Base automatically changed from nick/report-data-mask to main March 14, 2023 20:58
@nick-mobilecoin nick-mobilecoin merged commit c922013 into main Mar 14, 2023
@nick-mobilecoin nick-mobilecoin deleted the nick/cpu_svn branch March 14, 2023 20:59
@nick-mobilecoin
Copy link
Collaborator Author

@nick-mobilecoin merged this pull request with Graphite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Large PRs
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants