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 test_version during release process and simplify versions in README.md #1938

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

visr
Copy link
Member

@visr visr commented Nov 15, 2024

In the README.md we had version info like this:

release = "$tag"
commit = "$url/$short_commit"
branch = "$url/$short_name"
julia_version = "$julia_version"
core_version = "$version"

This cuts it down to just this:

version = "$tag"
commit = "$url/$short_commit"
branch = "$url/$short_name"

Where tag is of the form 2024.10.0-128-ge09eeb5 if it is after tag v2024.10.0, otherwise just 2024.10.0
The test_version just checks the form, because we bump the versions before doing a git tag, so they can be off by one version temporarily. I think that's simpler than introducing more difficult logic in set_version to handle this.

Fixes #1655

The `core_version` is like the `version`, but no info on dirty or post-tag status. The julia_version can be looked up from the commit, and is generally not very relevant for users.
@visr visr requested a review from evetion November 15, 2024 11:29
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.

Ignore test_version on version bump PR
1 participant