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

Use Wildcard When jvm_exclude Artifact is None #20802

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

Conversation

NGustafson
Copy link
Contributor

Fixes #20794

Update to_coord_str method to append * to the artifact coordinate string when a jvm_exclude artifact is None.

@huonw huonw requested a review from alonsodomin April 18, 2024 04:23
@huonw
Copy link
Contributor

huonw commented Apr 18, 2024

Awesome!

I'll let @alonsodomin shepherd this though for real, but from a quick skim, could we add a test for this, to be sure it continues to work in future?

@huonw huonw added the category:bugfix Bug fixes for released features label Apr 19, 2024
Copy link
Contributor

@alonsodomin alonsodomin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@NGustafson
Copy link
Contributor Author

Awesome!

I'll let @alonsodomin shepherd this though for real, but from a quick skim, could we add a test for this, to be sure it continues to work in future?

Sorry I've been busy recently. I can try to add a test next week.
I'm curious if there are any other tests that are currently resolving a dependency with Coursier. I'm guessing a test for this behavior might be flaky since it relies on fetching a 3rdparty artifact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:bugfix Bug fixes for released features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

jvm_exclude with Group Only Fails Parsing by Coursier
3 participants