microsoft / mu_feature_mm_supv

Project Mu - Feature Repo - MM Supervisor
https://microsoft.github.io/mu
Other
48 stars 29 forks source link

Fix override from CodeQL and cherry-picks from latest basecore #331

Closed kuqin12 closed 2 months ago

kuqin12 commented 2 months ago

Description

This change updated the override validation tags. It also integrates the cherry-pick commit from https://github.com/microsoft/mu_basecore/commit/4d722ac91608d3c4fbe807b418b77f2a6e279117.

For details on how to complete to complete these options and their meaning refer to CONTRIBUTING.md.

How This Was Tested

This was tested on pipeline and passed override check. Also booted on Q35 branch.

Integration Instructions

The change needs to pair with basecore commit https://github.com/microsoft/mu_basecore/commit/4d722ac91608d3c4fbe807b418b77f2a6e279117 or later.

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 0.50%. Comparing base (1900b9e) to head (ee68042).

Files with missing lines Patch % Lines
...mSupervisorPkg/Core/Services/MpService/SyncTimer.c 0.00% 8 Missing :warning:
MmSupervisorPkg/Library/BaseLibSysCall/String.c 0.00% 6 Missing :warning:
...mSupervisorPkg/Core/Services/MpService/MpService.c 0.00% 5 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #331 +/- ## ======================================== - Coverage 0.50% 0.50% -0.01% ======================================== Files 143 143 Lines 20921 20942 +21 Branches 60 60 ======================================== Hits 105 105 - Misses 20811 20832 +21 Partials 5 5 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/331/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | Coverage Δ | | |---|---|---| | [MmSupervisorPkg](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/331/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `0.50% <0.00%> (-0.01%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.