Skip to content

Conversation

shankarseal
Copy link
Collaborator

Description

Describe the purpose of and changes within this Pull Request.
Adds a version json file that is needed by the official build pipeline that is internal to Microsoft.

Testing

Do any existing tests cover this change? Are new tests needed?
No new tests.

Documentation

Is there any documentation impact for this change?
No documentation changes needed.

Installation

Is there any installer impact for this change?
No installation changes needed.

@saxena-anurag saxena-anurag enabled auto-merge (squash) October 17, 2025 00:49
@saxena-anurag saxena-anurag merged commit 91050bb into microsoft:release/1.0 Oct 17, 2025
111 checks passed
Copy link
Collaborator

@dthaler dthaler left a comment

Choose a reason for hiding this comment

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

I think there is a problem with this PR, but I didn't get a chance to comment before it was merged.

  1. The PR description says there is no impact on documentation. docs\ReleaseProcess.md has a section entitled "Updating the Product Version" that says you just need to run .\scripts\update-product-version.ps1 X Y Z.
  2. update-product-version.ps1 was not updated and so is not aware of version.json.

Taken together, this means the documentation is wrong and this is a regression.

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