Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Mar 25, 2025

We have been using SemVer to handle version updates in provider documentation breeze commands, which started to break at the moment we started to have pre-release packages - Puthon packaging follows PEP-440 and it is implemented in packaging.version module.

This module does not have all the nice features semver has (like bumping the major/minor/patchlevel versions so we had to implement it ourselves (packaging Versions are immutable and we need to recreate the version after bump.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@uranusjr
Copy link
Member

packaging.version.InvalidVersion: Invalid version: "3.10.5('b', 1)"

Looks like it’s still not parsed right.

We have been using SemVer to handle version updates in provider
documentation breeze commands, which started to break at the moment
we started to have pre-release packages - Puthon packaging follows
PEP-440 and it is implemented in packaging.version module.

This module does not have all the nice features semver has (like
bumping the major/minor/patchlevel versions so we had to implement
it ourselves (packaging Versions are immutable and we need to
recreate the version after bump.
@potiuk potiuk force-pushed the change_version_handling_from_semver_to_packaging branch from 42590e8 to 8511da3 Compare March 25, 2025 17:12
@potiuk
Copy link
Member Author

potiuk commented Mar 25, 2025

packaging.version.InvalidVersion: Invalid version: "3.10.5('b', 1)"

Looks like it’s still not parsed right.

Indeed . Should be fixed. kinda surprising to see tuple there, and int in dev , but makes sense for sorting and the like

@potiuk
Copy link
Member Author

potiuk commented Mar 25, 2025

ok. Looks good. Merging

@potiuk potiuk merged commit 40bc3a5 into apache:main Mar 25, 2025
88 checks passed
@potiuk potiuk deleted the change_version_handling_from_semver_to_packaging branch March 25, 2025 17:49
pankajkoti pushed a commit to astronomer/airflow that referenced this pull request Mar 28, 2025
apache#48249)

We have been using SemVer to handle version updates in provider
documentation breeze commands, which started to break at the moment
we started to have pre-release packages - Puthon packaging follows
PEP-440 and it is implemented in packaging.version module.

This module does not have all the nice features semver has (like
bumping the major/minor/patchlevel versions so we had to implement
it ourselves (packaging Versions are immutable and we need to
recreate the version after bump.
shubham-pyc pushed a commit to shubham-pyc/airflow that referenced this pull request Apr 2, 2025
apache#48249)

We have been using SemVer to handle version updates in provider
documentation breeze commands, which started to break at the moment
we started to have pre-release packages - Puthon packaging follows
PEP-440 and it is implemented in packaging.version module.

This module does not have all the nice features semver has (like
bumping the major/minor/patchlevel versions so we had to implement
it ourselves (packaging Versions are immutable and we need to
recreate the version after bump.
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
apache#48249)

We have been using SemVer to handle version updates in provider
documentation breeze commands, which started to break at the moment
we started to have pre-release packages - Puthon packaging follows
PEP-440 and it is implemented in packaging.version module.

This module does not have all the nice features semver has (like
bumping the major/minor/patchlevel versions so we had to implement
it ourselves (packaging Versions are immutable and we need to
recreate the version after bump.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants