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 testcase for classifier with transitive dependencies #1698

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

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Feb 21, 2024

Currently if a dependency is specified the transitive items seem to be not correctly collected, at laest there is one artifact (ehcache) that has a transitive dependency that is invalid (javax.xml) and one can select a different classifier (jakarta) to use the jakarta ones but this seem not to happen at the moment.

This adds a testcase to illustrate the problem.

Currently if a dependency is specified  the transitive items seem to be
not correctly collected, at laest there is one artifact (ehcache) that
has a transitive dependency that is invalid (javax.xml) and one can
select a different classifier (jakarta) to use the jakarta ones but this
seem not to happen at the moment.

This adds a testcase to illustrate the problem.
Copy link

Test Results

  214 files  +107    214 suites  +107   16m 19s ⏱️ + 6m 8s
  666 tests +  1    654 ✅  -   1  10 💤 ± 0  1 ❌ +1  1 🔥 +1 
1 332 runs  +667  1 307 ✅ +653  22 💤 +11  2 ❌ +2  1 🔥 +1 

For more details on these failures and errors, see this check.

Results for commit c200cea. ± Comparison against base commit 8789e4c.

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.

1 participant