Skip to content

windows: ensure app manifest version is formatted correctly - #2376

Merged
mjcheetham merged 1 commit into
git-ecosystem:vnextfrom
mjcheetham:fix-winmanifest
Jul 13, 2026
Merged

mjcheetham merged 1 commit into
git-ecosystem:vnextfrom
mjcheetham:fix-winmanifest

Conversation

@mjcheetham

Copy link
Copy Markdown
Contributor

Windows requires the version property in an application manifest to be of the format "x.y.z.w" which specific rules about the min/max values for each component (16-bit numbers).

Use System.Version to parse the version and normalise/validate before writing the manifest file.

Windows requires the version property in an application manifest to be
of the format "x.y.z.w" which specific rules about the min/max values
for each component (16-bit numbers).

Use `System.Version` to parse the version and normalise/validate before
writing the manifest file.

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
@mjcheetham
mjcheetham requested a review from dscho July 13, 2026 10:05
@mjcheetham
mjcheetham requested a review from a team as a code owner July 13, 2026 10:05
@mjcheetham mjcheetham added gcm3.0 Targets the next major release of Git Credential Manager - v3.0 bug A bug in Git Credential Manager platform:windows Specific to the Windows platform labels Jul 13, 2026

@dscho dscho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me!

@mjcheetham
mjcheetham merged commit d856393 into git-ecosystem:vnext Jul 13, 2026
25 checks passed
@mjcheetham
mjcheetham deleted the fix-winmanifest branch July 13, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A bug in Git Credential Manager gcm3.0 Targets the next major release of Git Credential Manager - v3.0 platform:windows Specific to the Windows platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants