-
Notifications
You must be signed in to change notification settings - Fork 76
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
Ability to pack test scoped jars #110 #324
Conversation
Interesting. Just from curiosity, could you tell me how you use the packed test code? |
It seems this forces the user to always select Compile or Test target for the pack command. Any idea so as not to introduce such a breaking change? |
We use ZIO-test, therefore each test suite is a runnable main class. Also we have a custom RunAllSpecs main that uses reflection to run all test suites in our package in the classpath. Then we simply add
Running |
OK. Let me merge. Will release a new version soon |
@Marc-Ducret Released 0.17, but it seems projects using sbt-pack is broken like this. wvlet/airframe#2584 I guess some tests are missing |
ok. I've found a fix. If |
This pr bumps sbt-pack to the latest 0.17 and also accounts for a breaking change that was introduced in xerial/sbt-pack#324 meaning that `pack` now must be scoped.
This pr bumps sbt-pack to the latest 0.17 and also accounts for a breaking change that was introduced in xerial/sbt-pack#324 meaning that `pack` now must be scoped.
Using Def.derive allows to propagate Compile/Test configurations.
Therefore
Compile / pack
packs without the tests andTest / pack
packs with the tests.