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

python: switch to static versioning for simplicity #1419

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

lidavidm
Copy link
Member

@lidavidm lidavidm commented Jan 2, 2024

We tried to derive versions from Git. But since we release off of a branch, this meant that all non-branch builds would have version 0.0.0. This breaks things like Polars that rely on checking the installed version.

Instead, since we already bump versions as part of the release process, just hardcode the version and drop most of miniver.

Fixes #1363.

We tried to derive versions from Git.  But since we release off of
a branch, this meant that all non-branch builds would have version
0.0.0.  This breaks things like Polars that rely on checking the
installed version.

Instead, since we already bump versions as part of the release
process, just hardcode the version and drop most of miniver.

Fixes apache#1363.
Copy link

github-actions bot commented Jan 2, 2024

⚠️ Please follow the Conventional Commits format in CONTRIBUTING.md for PR titles.

@github-actions github-actions bot added this to the ADBC Libraries 0.9.0 milestone Jan 2, 2024
@lidavidm lidavidm marked this pull request as ready for review January 2, 2024 17:23
Copy link
Contributor

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

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

lgtm. We could also take a look at tools like versioneer

@lidavidm
Copy link
Member Author

lidavidm commented Jan 2, 2024

I had originally picked miniver as a simpler version of versioneer; if they work roughly the same they probably have the same failure mode here

@lidavidm lidavidm merged commit 76ab8cb into apache:main Jan 3, 2024
49 checks passed
@lidavidm lidavidm deleted the gh1363 branch January 3, 2024 13:29
@jorisvandenbossche
Copy link
Member

I think bumping manually is certainly fine here (and I also don't do the releasing, so don't have an opinion on it), but just a small comment on:

But since we release off of a branch, this meant that all non-branch builds would have version 0.0.0.

I (help) maintain other projects that also do releases from a branch and do use versioneer. That in itself is not a problem, but that only means you need to tag the main branch as well, and not only the release branch (eg in pandas, when we make the release branch for eg 2.2.x, we tag the next commit on the main branch as 2.3.0.dev to ensure the main dev version is correctly "higher")

@lidavidm
Copy link
Member Author

lidavidm commented Jan 8, 2024

Ah. That makes sense, though it feels a little synthetic.

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.

ci: fix failing builds
3 participants