Skip to content

Conversation

@makubacki
Copy link
Member

Description

Adds a plugin that is not active by default but can be opted into that validates git repositories against remote commit criteria before the build starts. It allows you to define rules that check whether the current HEAD commit in specified repositories is present or absent in specified remote URLs.

This is useful for enforcing that your local repositories are using commits from the correct upstream sources, and not from incompatible forks or remotes. For example, some workspaces might use a "public" and an "internal" upstream and need to verify that both work as expected and are set to the appropriate upstream when used for a given platform build.

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

How This Was Tested

  • Verify that the plugin does not run unless the scope is set
  • Verify JSON and YAML files can be parsed
  • Verify repo check logic returns expected results

Integration Instructions

  • See the plugin readme file

Adds a plugin that is not active by default but can be opted into
that validates git repositories against remote commit criteria
before the build starts. It allows you to define rules that check
whether the current HEAD commit in specified repositories is
present or absent in specified remote URLs.

This is useful for enforcing that your local repositories are using
commits from the correct upstream sources, and not from incompatible
forks or remotes. For example, some workspaces might use a "public"
and an "internal" upstream and need to verify that both work as
expected and are set to the appropriate upstream when used for a
given platform build.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@codecov-commenter
Copy link

codecov-commenter commented Oct 27, 2025

Codecov Report

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

Additional details and impacted files
@@                Coverage Diff                @@
##             release/202502    #1546   +/-   ##
=================================================
  Coverage                  ?    2.22%           
=================================================
  Files                     ?     1461           
  Lines                     ?   380909           
  Branches                  ?     4583           
=================================================
  Hits                      ?     8477           
  Misses                    ?   372359           
  Partials                  ?       73           
Flag Coverage Δ
MdeModulePkg 1.55% <ø> (?)
MdePkg 5.46% <ø> (?)
NetworkPkg 0.55% <ø> (?)
PolicyServicePkg 30.42% <ø> (?)
UefiCpuPkg 4.86% <ø> (?)
UnitTestFrameworkPkg 11.40% <ø> (?)

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.

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.

2 participants