Skip to content

support semver 3.x - #45

Open
AlonNR wants to merge 1 commit into
azimuth-cloud:mainfrom
AlonNR:support-semver-3
Open

support semver 3.x#45
AlonNR wants to merge 1 commit into
azimuth-cloud:mainfrom
AlonNR:support-semver-3

Conversation

@AlonNR

@AlonNR AlonNR commented Jul 24, 2026

Copy link
Copy Markdown

PR #41 explicitly restricts semver to versions >=2.9.1,<3 while latest release is 3.0.4

this PR extends supported range to <4
and replaces one use of the deprecated semver.parse_version_info with semver.VersionInfo.parse which is used by the rest of this project.

PR azimuth-cloud#41 explicitly restricts semver to versions >=2.9.1,<3 while latest release is 3.0.4

this PR extends supported range to <4
and replaces one use of the deprecated `semver.parse_version_info` with `semver.VersionInfo.parse` which is used by the rest of this project.
Comment thread pyhelm3/command.py
)
version_str = proc.stdout.decode().removeprefix("v")
return semver.parse_version_info(version_str)
return semver.VersionInfo.parse(version_str)

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.

This is the wrong replacement according to the upstream migration guide https://python-semver.readthedocs.io/en/latest/migration/replace-deprecated-functions.html

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

hi thanks for the feedback,
you can see in https://github.com/python-semver/python-semver/blob/master/src/semver/version.py#L762
that VersionInfo is an alias of Version

if you prefer, I can align all usages of semver.VersionInfo.parse in the repo to use semver.Version.parse instead

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@irt-shpc how do you wish to proceed?

@AlonNR
AlonNR requested a review from irt-shpc July 24, 2026 15:37
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