mu_basecore icon indicating copy to clipboard operation
mu_basecore copied to clipboard

Add deprecation warning support to OverrideValidation plugin

Open NishanthSanjeevi opened this issue 1 year ago • 2 comments

Description

Added deprecation warning support to the existing Override validation plugin/tool.

  • [ ] Impacts functionality?
    • Functionality - All libraries/drivers that are no longer used should add a Deprecation warning
  • [ ] Impacts security?
    • Security - N/A
  • [ ] Breaking change?
    • Breaking change - Will anyone consuming this change experience a break in build or boot behavior? No
  • [ ] Includes tests? No
  • [ ] Includes documentation? Added documentation for how to use the Deprecation warnings module

How This Was Tested

Added the Deprecation warnings to the INFs and a warning was thrown when a deprecated module was part of the DSC

Integration Instructions

N/A

NishanthSanjeevi avatar Feb 16 '24 04:02 NishanthSanjeevi

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 1.35%. Comparing base (856d8a4) to head (161643d).

Additional details and impacted files
@@               Coverage Diff               @@
##           release/202311     #742   +/-   ##
===============================================
  Coverage            1.35%    1.35%           
===============================================
  Files                1304     1304           
  Lines              333854   333854           
  Branches             5103     5103           
===============================================
  Hits                 4512     4512           
  Misses             329259   329259           
  Partials               83       83           
Flag Coverage Δ
MdeModulePkg 0.68% <ø> (ø)
MdePkg 5.40% <ø> (ø)
NetworkPkg 0.55% <ø> (ø)
PolicyServicePkg 30.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

codecov-commenter avatar Feb 16 '24 04:02 codecov-commenter

@NishanthSanjeevi, please resolve the final open conversations and we can proceed to merging the PR.

makubacki avatar May 29 '24 20:05 makubacki

Hi @NishanthSanjeevi, it looks like changes to the readme are causing MarkdownLint to fail it's CI check. Please review and resolve. Thanks!

Javagedes avatar Jul 24 '24 15:07 Javagedes