Skip to content

fix(build): remove GenerateAssemblyInfo=false so assemblies carry version info - #924

Open
Hardik K (hardikkaurani) wants to merge 1 commit into
microsoft:mainfrom
hardikkaurani:fix/enable-assembly-info-versioning
Open

Hardik K (hardikkaurani) wants to merge 1 commit into
microsoft:mainfrom
hardikkaurani:fix/enable-assembly-info-versioning

Conversation

@hardikkaurani

Copy link
Copy Markdown

Description

Fixes #852: Microsoft.OpenApi.OData.Reader.dll ships unversioned (0.0.0.0) since the 2026-01-16 releases.

Cause

In commit 0fb398e9 (PR #766), legacy MSBuild versioning files (tool/versioning.props, etc.) were removed, but <GenerateAssemblyInfo>false</GenerateAssemblyInfo> remained in src/Build.props. With assembly-info generation disabled and the legacy toolchain gone, no version attributes (AssemblyVersion, AssemblyFileVersion, AssemblyInformationalVersion) were stamped on the compiled assemblies, leaving them at 0.0.0.0.

Changes Made

  • Removed <GenerateAssemblyInfo>false</GenerateAssemblyInfo> and the stale comment from src/Build.props.
  • This enables standard .NET SDK assembly info generation using the <Version> already defined in Directory.Build.props.

Fixes #852

cc Gavin Barron (@gavinbarron) Vincent Biret (@baywet)

@hardikkaurani
Hardik K (hardikkaurani) requested a review from a team as a code owner October 19, 2026 20:11
Copilot AI balanced review requested due to automatic review settings October 19, 2026 20:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@hardikkaurani

Copy link
Copy Markdown
Author

Hi Gavin Barron (@gavinbarron) Vincent Biret (@baywet), submitted this PR to address #852 by removing <GenerateAssemblyInfo>false</GenerateAssemblyInfo> left behind in src/Build.props. This allows .NET SDK to automatically stamp version attributes (AssemblyVersion, FileVersion, ProductVersion) using the <Version> defined in Directory.Build.props. Ready for review, thank you!

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.

Microsoft.OpenApi.OData.Reader.dll ships unversioned (0.0.0.0) since the 2026-01-16 releases

2 participants