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

Handle version range in dependencies for target locations #1964

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

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Mar 18, 2025

Currently if a dependency uses a version range this fails to resolve the target as maven-resolver tries to use the version "as-is"

This now adds a testcase and a fix to resolve to the highest possible version in such case.

See

Copy link

github-actions bot commented Mar 18, 2025

Test Results

0 files   -   324  0 suites   - 324   0s ⏱️ - 1h 11m 32s
0 tests  -   686  0 ✅  -   663  0 💤  - 20  0 ❌  - 2 
0 runs   - 2 058  0 ✅  - 1 995  0 💤  - 60  0 ❌  - 2 

Results for commit 46c0304. ± Comparison against base commit fca5d73.

♻️ This comment has been updated with latest results.

Currently if a dependency uses a version range this fails to resolve the
target as maven-resolver tries to use the version "as-is"

THis now adds a testcase and a fix to resolve to the highest possible
version in such case.
@laeubi laeubi force-pushed the maven_target_verion_ranges branch from 5c4fd8c to 46c0304 Compare March 18, 2025 16:01
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.

None yet

1 participant