Skip to content

Remove unnecessary flag switch that causes official build to break.#3700

Merged
MZOLN merged 2 commits into
masterfrom
marcinzo/disable_explicit_packagegen
Feb 3, 2026
Merged

Remove unnecessary flag switch that causes official build to break.#3700
MZOLN merged 2 commits into
masterfrom
marcinzo/disable_explicit_packagegen

Conversation

@MZOLN

@MZOLN MZOLN commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

This seem unnecessary and causes overrides of binaries while signing.

@MZOLN MZOLN requested a review from a team as a code owner February 3, 2026 22:00
@MZOLN MZOLN merged commit e901171 into master Feb 3, 2026
5 checks passed
@MZOLN MZOLN deleted the marcinzo/disable_explicit_packagegen branch February 3, 2026 22:37

@jmprieur jmprieur left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's up the version please 4.3.0 is out
and please follow the release process

Comment thread Directory.Build.props
<!-- This needs to be greater than or equal to the validation baseline version. The conditional logic around TargetNetNext is there
to avoid NU5104 for packing a release version library with prerelease deps. By adding preview to it, that warning is avoided.
-->
<MicrosoftIdentityWebVersion Condition="'$(MicrosoftIdentityWebVersion)' == ''">4.3.1-preview</MicrosoftIdentityWebVersion>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It should be 4.3.1
IT should never have -preview1 in this file.

@copilot please update the agent.md to check that

<ProjectGuid>{74737C0A-25E6-4E2C-B3AA-F7EECB7A07C7}</ProjectGuid>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<PackageReadmeFile>README.md</PackageReadmeFile>
<GeneratePackageOnBuild>True</GeneratePackageOnBuild>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What's the issue with this one?

This was referenced Mar 1, 2026
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.

4 participants