Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Version Comparison Error in YamlCreate #244669

Merged
merged 1 commit into from
Apr 2, 2025
Merged

Conversation

Trenly
Copy link
Contributor

@Trenly Trenly commented Apr 2, 2025

Compare versions as versions and not as strings

Checklist for Pull Requests

Fixes an error where Zip files can't be selected in YamlCreate because 1.10.0 is lexicographically less than 1.4.0 by explicitly casting to versions.

@denelon


Microsoft Reviewers: Open in CodeFlow

Compare versions as versions and not as strings
@Trenly Trenly requested review from denelon and a team as code owners April 2, 2025 20:44
@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Attention This work item needs to be reviewed by a member of the core team. label Apr 2, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Apr 2, 2025
@mdanish-kh
Copy link
Contributor

[Policy] Needs-Manual-Merge

@denelon denelon merged commit a275b4b into microsoft:master Apr 2, 2025
2 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Manual-Merge The pull request requires a manual merge from a repository maintainer Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Attention This work item needs to be reviewed by a member of the core team. labels Apr 2, 2025
@Trenly Trenly deleted the FixCompare branch April 2, 2025 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Moderator-Approved One of the Moderators has reviewed and approved this PR Needs-Attention This work item needs to be reviewed by a member of the core team. Needs-Manual-Merge The pull request requires a manual merge from a repository maintainer Project-File
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants