Skip to content

Conversation

aircraft-cerier
Copy link

Description

Adds support for pessimistic operations where just MAJOR version is provided:
~> 18

Related Issue

#162

How Has This Been Tested?

tested using go test

added 3 test cases:

		{"~> 1", "1.9", true},
		{"~> 1", "2.0", false},
		{"~> 1", "2", false},

@aircraft-cerier aircraft-cerier requested a review from a team as a code owner July 10, 2025 23:31
{"1.0", "1.0.0", true},
{"~> 1", "1.9", true},
{"~> 1", "2.0", false},
{"~> 1", "2", false},

Choose a reason for hiding this comment

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

this one shouldn't be possible right?

{"= 1.0", "1.1.5", false},
{"= 1.0", "1.0.0", true},
{"1.0", "1.0.0", true},
{"~> 1", "1.9", true},

Choose a reason for hiding this comment

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

I think this case is already covered by existing tests 🤔

Copy link

@davidmathis davidmathis left a comment

Choose a reason for hiding this comment

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

nits - but overall lgtm

@aircraft-cerier
Copy link
Author

@mukeshjc @radeksimko would it be possible to get a review for this?

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.

2 participants