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

[CHERRY-PICK] Added missing fix for ProductDataFormatter missed from 202311 (#1195) #1196

Conversation

kenlautner
Copy link
Contributor

Description

A fix for the ProductDataFormatter script was missed during the transition from 202311 to 202405. This just adds it back.

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

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Tested on physical hardware. Product Data is correctly being found.

Integration Instructions

N/A

…oft#1195)

## Description

A fix for the ProductDataFormatter script was missed during the
transition from 202311 to 202405. This just adds it back.

For details on how to complete these options and their meaning refer to
[CONTRIBUTING.md](https://github.com/microsoft/mu/blob/HEAD/CONTRIBUTING.md).

- [ ] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?

## How This Was Tested

Tested on physical hardware.  Product Data is correctly being found.

## Integration Instructions

N/A
@kenlautner kenlautner changed the title Added missing fix for ProductDataFormatter missed from 202311 (#1195) [CHERRY-PICK] Added missing fix for ProductDataFormatter missed from 202311 (#1195) Oct 24, 2024
@github-actions github-actions bot added language:python Pull requests that update Python code impact:non-functional Does not have a functional impact labels Oct 24, 2024
@codecov-commenter
Copy link

codecov-commenter commented Oct 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 1.59%. Comparing base (bf73b8a) to head (3d8a3c5).

Additional details and impacted files
@@               Coverage Diff               @@
##           release/202405    #1196   +/-   ##
===============================================
  Coverage            1.59%    1.59%           
===============================================
  Files                1448     1448           
  Lines              362502   362502           
  Branches             5632     5632           
===============================================
  Hits                 5776     5776           
  Misses             356619   356619           
  Partials              107      107           
Flag Coverage Δ
MdeModulePkg 0.68% <ø> (ø)
MdePkg 5.42% <ø> (ø)
NetworkPkg 0.55% <ø> (ø)
PolicyServicePkg 30.41% <ø> (ø)
UefiCpuPkg 4.73% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kenlautner kenlautner enabled auto-merge (rebase) October 25, 2024 16:50
@apop5
Copy link
Contributor

apop5 commented Oct 25, 2024

This needs to go into dev as well, correct?

@kenlautner
Copy link
Contributor Author

kenlautner commented Oct 25, 2024

This needs to go into dev as well, correct?

This went into DEV already. I cherry-picked it over for this PR.

@kenlautner kenlautner merged commit 79a2924 into microsoft:release/202405 Oct 25, 2024
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:non-functional Does not have a functional impact language:python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants