Skip to content

chore: remove SNAPSHOT from version#3600

Merged
westonpace merged 2 commits intolance-format:mainfrom
westonpace:chore/remove-snapshot-java-ci
Mar 26, 2025
Merged

chore: remove SNAPSHOT from version#3600
westonpace merged 2 commits intolance-format:mainfrom
westonpace:chore/remove-snapshot-java-ci

Conversation

@westonpace
Copy link
Copy Markdown
Member

Reverts part of #3546 which added -SNAPSHOT to the versions. Currently the CI build system does not publish Java artifacts on pre-releases. There is also nothing in the build script to remove the -SNAPSHOT designation from the version. As a result the publish failed.

Currently, CI requires the version specifier point to the next stable version that will be released. This restores that so the next stable release can succeed.

@yanghua
Copy link
Copy Markdown
Collaborator

yanghua commented Mar 25, 2025

Yes, I found this issue and opened a pr: #3597.

Want to know why we do not cut a branch for the release candidate?

@westonpace westonpace merged commit 5891b1a into lance-format:main Mar 26, 2025
8 checks passed
eddyxu pushed a commit that referenced this pull request Mar 26, 2025
Reverts part of #3546 which added `-SNAPSHOT` to the versions. Currently
the CI build system does not publish Java artifacts on pre-releases.
There is also nothing in the build script to remove the `-SNAPSHOT`
designation from the version. As a result the publish failed.

Currently, CI requires the version specifier point to the next stable
version that will be released. This restores that so the next stable
release can succeed.
Jay-ju pushed a commit to Jay-ju/lance that referenced this pull request Mar 28, 2025
Reverts part of lance-format#3546 which added `-SNAPSHOT` to the versions. Currently
the CI build system does not publish Java artifacts on pre-releases.
There is also nothing in the build script to remove the `-SNAPSHOT`
designation from the version. As a result the publish failed.

Currently, CI requires the version specifier point to the next stable
version that will be released. This restores that so the next stable
release can succeed.
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.

2 participants