Closed Javagedes closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Please upload report for BASE (
release/202405@63fc132
). Learn more about missing BASE report.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
We can revert this in the same PR: https://github.com/microsoft/mu_basecore/commit/7281ec2297. Also, I don't think this needs to be a breaking change. We do not believe any platform will break because of this, nor are there integration steps to fix it, so this would go in general category of shouldn't break but if it does, let us know.
I will approve. Waiting for @Javagedes to get back on an additional platform build that I recommended to test.
We can revert this in the same PR: 7281ec2297. Also, I don't think this needs to be a breaking change. We do not believe any platform will break because of this, nor are there integration steps to fix it, so this would go in general category of shouldn't break but if it does, let us know.
Sounds good. I was being overly pessamistic with marking it as a breaking change. I'll undo that portion and also revert the PR you mentioned, as a REBASE&FF
@makubacki The additional platform was confirmed to successfully build on windows for DEBUG and RELEASE targets.
Description
This reverts the following commits:
6f5dd604f8992346983ddea926d3adbce81dae53
01f1b2da98675e992e7f723b13e859a9a50be40d
[ ] Impacts functionality?
[ ] Impacts security?
[] Breaking change?
[ ] Includes tests?
[ ] Includes documentation?
How This Was Tested
Confirmed this does not affect multiple platform builds using VS compiler
Integration Instructions
N/A