Merge latest Library.Template

[AArnott]

Merges the latest Library.Template microbuild branch.

[Copilot]

Pull request overview

This PR syncs this repo with the latest Library.Template microbuild branch, updating shared build/test infrastructure so CI artifacts and documentation generation align with the template’s current expectations.

Changes:

• Filter test artifact collection to avoid uploading crash/hang dump files except when they’re in the TRX attachment (In) directory.
• Bump Microsoft Testing Platform package version to 2.3.3.
• Update the docs GitHub Actions workflow to skip installing the NuGet credential provider during init.ps1.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
tools/artifacts/testResults.ps1	Adds filtering around .dmp test artifacts (keep only those in TRX attachments).
Directory.Packages.props	Updates centralized Microsoft Testing Platform version from 2.3.2 to 2.3.3.
.github/workflows/docs.yml	Adjusts docs workflow prereq install to pass -NoNuGetCredProvider.

Comments suppressed due to low confidence (1)

tools/artifacts/testResults.ps1:21

• Same issue as above: this regex only matches \\In\\ and will not match /In/ paths on non-Windows agents, unintentionally filtering out crash/hang dumps. Prefer a separator-agnostic regex and case-insensitive extension check.

        Where-Object { $_.Extension -ne '.dmp' -or $_.FullName -match '\\In\\' }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (2)

tools/artifacts/testResults.ps1:21

• Similar to the legacy TestResults block, Get-ChildItem $testlogsPath -Recurse returns directories as well as files, and the Where-Object filter will keep directories (they have an empty .Extension). These directories are later ignored by the staging script, but they still add unnecessary enumeration overhead and can mask the "No files found" warning. Limit this to files only.

    $result[$testlogsPath] = Get-ChildItem $testlogsPath -Recurse |
        Where-Object { $_.Extension -ne '.dmp' -or $_.FullName -match '[/\\]In[/\\]' }

tools/artifacts/testResults.ps1:13

• The current pipeline enumerates directories and then runs a recursive file search for each directory (Get-ChildItem ... -Recurse -Directory | Get-ChildItem -Recurse -File). With nested TestResults folders this repeats work and can return the same file multiple times (once per ancestor directory), which can slow artifact collection and cause redundant staging work. A single recursive file enumeration is sufficient.

This issue also appears on line 20 of the same file.

    $result[$testRoot] = Get-ChildItem $legacyTestResults -Recurse -Directory |
        Get-ChildItem -Recurse -File |
        Where-Object { $_.Extension -ne '.dmp' -or $_.FullName -match '[/\\]In[/\\]' }