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

Add support for snapshotted Maven artifacts #974

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmmv
Copy link

@jmmv jmmv commented Oct 10, 2023

Artifacts with snapshots follow a different naming scheme than non-snapshoted ones. For snapshotted artifacts, the version component in the URL is the generic *-SNAPSHOT name, whereas the version component in the file name is the specific version number.

Fixes #901.

@jmmv
Copy link
Author

jmmv commented Oct 11, 2023

Note that the conflict in the prebuilt tool is expected, because this PR is based off 5.3 and not main. I don't think you'll want to trust the artifact I built anyway, so I can remove it from the PR when the rest of it is reviewed.

Artifacts with snapshots follow a different naming scheme than
non-snapshoted ones.  For snapshotted artifacts, the version component
in the URL is the generic *-SNAPSHOT name, whereas the version component
in the file name is the specific version number.

Fixes bazel-contrib#901.
@jmmv jmmv force-pushed the issue-901-snapshots branch from 080a5f7 to db0aeb8 Compare October 12, 2023 16:21
@tharakadesilva
Copy link

Hi folks, any update on this?

@stepango
Copy link

Facing same issue with 6.0, any chance this change will get into 6.1?

@peakschris
Copy link

@jin @Wyverald @shs96c @cheister I'm also facing this issue. Any chance this PR could be taken forward?

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.

SNAPSHOT versions not working in LockFileConverter
4 participants