Skip to content

SecurityPkg: Add missing TpmTestApp markdown information#1768

Merged
Raymond-MS merged 2 commits intomicrosoft:release/202511from
Raymond-MS:personal/rdiaz/TpmTestAppMdUpdate
Apr 17, 2026
Merged

SecurityPkg: Add missing TpmTestApp markdown information#1768
Raymond-MS merged 2 commits intomicrosoft:release/202511from
Raymond-MS:personal/rdiaz/TpmTestAppMdUpdate

Conversation

@Raymond-MS
Copy link
Copy Markdown
Contributor

@Raymond-MS Raymond-MS commented Apr 16, 2026

Description

Added missing information to the TpmTestApp markdown document.

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

How This Was Tested

N/A

Integration Instructions

N/A

@Raymond-MS Raymond-MS requested review from cfernald and kuqin12 April 16, 2026 18:28
@Raymond-MS Raymond-MS force-pushed the personal/rdiaz/TpmTestAppMdUpdate branch from 45d2274 to 8dec62e Compare April 16, 2026 18:29
@mu-automation
Copy link
Copy Markdown
Contributor

mu-automation bot commented Apr 16, 2026

✅ QEMU Validation Passed

Source Dependencies

Repository Commit
mu_basecore 623d1c9
mu_tiano_platforms f61452e

Results

Platform Target Build Boot Overall Boot Time Build Logs Boot Logs
Q35 DEBUG ✅ success ✅ success 0m 17s Build Logs Boot Logs
SBSA DEBUG ✅ success ✅ success 0m 31s Build Logs Boot Logs

Workflow run: https://github.com/microsoft/mu_basecore/actions/runs/24582839709

This comment was automatically generated by the Mu QEMU PR Validation workflow.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (release/202511@3018f1b). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff                @@
##             release/202511   #1768   +/-   ##
================================================
  Coverage                  ?   1.59%           
================================================
  Files                     ?     160           
  Lines                     ?   37728           
  Branches                  ?     172           
================================================
  Hits                      ?     601           
  Misses                    ?   37127           
  Partials                  ?       0           
Flag Coverage Δ
SecurityPkg 1.59% <ø> (?)

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cfernald cfernald requested a review from os-d April 17, 2026 17:40
Copy link
Copy Markdown
Contributor

@os-d os-d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add [SQUASH ON REBASE] to the commit title when merging and put the commit hash of the original commit that added this (i.e. should be squashed with on rebase?

Also, if this test app is ready to go, can you upstream to edk2?

@Raymond-MS Raymond-MS force-pushed the personal/rdiaz/TpmTestAppMdUpdate branch from 8dec62e to 623d1c9 Compare April 17, 2026 19:19
@Raymond-MS Raymond-MS merged commit dff1fa7 into microsoft:release/202511 Apr 17, 2026
56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants