Skip to content

Centralize app version info in project file for maintainability - #2055

Merged
Marcel W. (marcelwgn) merged 10 commits into
microsoft:mainfrom
AndrewKeepCoding:centralizing-app-version
Dec 17, 2025
Merged

Centralize app version info in project file for maintainability#2055
Marcel W. (marcelwgn) merged 10 commits into
microsoft:mainfrom
AndrewKeepCoding:centralizing-app-version

Conversation

@AndrewKeepCoding

Copy link
Copy Markdown
Contributor

Description

This PR centralizes the app version management by introducing a Version property in the app project file.

Motivation and Context

Currently, we are manually updating these files to bump the app version, which is error-prone and easy to miss.

  • Package.appxmanifest
  • Package.Dev.appxmanifest
  • AssemblyInfo.cs

Closes #2054.

How Has This Been Tested?

Ran the app and confirmed the changes.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Comment thread WinUIGallery/WinUIGallery.csproj Outdated
@marcelwgn

Copy link
Copy Markdown
Contributor

/azp run

@marcelwgn
Marcel W. (marcelwgn) enabled auto-merge (squash) October 28, 2025 14:48
@marcelwgn

Copy link
Copy Markdown
Contributor

Thank you for fixing this and sorry for the delay in reviewing this.

@AndrewKeepCoding

Copy link
Copy Markdown
Contributor Author

Thanks Marcel W. (@marcelwgn) !
It looks like the CI is failing on WinUI-Gallery-PRValidation, but I don't have access to view the details. Would you mind taking a look when you get a chance?

@marcelwgn

Copy link
Copy Markdown
Contributor

Looks like something funny with the nuget restore, let me retry

@marcelwgn

Copy link
Copy Markdown
Contributor

/azp run

@marcelwgn

Copy link
Copy Markdown
Contributor

Nuget is still complaining, I will take a look another time. The changes in here dont seem like they should cause nuget restore to fail

@marcelwgn

Copy link
Copy Markdown
Contributor

/azp run

@marcelwgn

Copy link
Copy Markdown
Contributor

/azp run

@marcelwgn
Marcel W. (marcelwgn) merged commit 57a6249 into microsoft:main Dec 17, 2025
2 checks passed
@AndrewKeepCoding
Andrew KeepCoding (AndrewKeepCoding) deleted the centralizing-app-version branch December 17, 2025 23:55
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.

Centralize app version

2 participants