Skip to content

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Sep 3, 2025

SiteExtensions build consumes packages that it expects to be real-signed. The unofficial pipeline does test-signing instead, so we should ignore that error there.

Test build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2785826&view=results

@Copilot Copilot AI review requested due to automatic review settings September 3, 2025 23:12
@wtgodbe wtgodbe requested a review from a team as a code owner September 3, 2025 23:13
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the unofficial build pipeline by suppressing a NuGet package signing validation error. The SiteExtensions build expects real-signed packages, but the unofficial pipeline uses test-signing, causing NU3027 warnings that should be ignored in development builds.

Key changes:

  • Add conditional suppression of NU3027 warning for development builds
  • Configure unofficial build pipeline to use development build mode

Reviewed Changes

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

File Description
src/SiteExtensions/Directory.Build.props Adds conditional NoWarn property to suppress NU3027 signing validation errors when _DevBuild is true
.azure/pipelines/ci-unofficial.yml Sets _DevBuild=true parameter for the SiteExtension build step in unofficial pipeline

@github-actions github-actions bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Sep 3, 2025
@wtgodbe wtgodbe merged commit 3aa4a1d into main Sep 4, 2025
31 checks passed
@wtgodbe wtgodbe deleted the wtgodbe/FixUnof branch September 4, 2025 02:35
@dotnet-policy-service dotnet-policy-service bot added this to the 11.0-preview1 milestone Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant