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

Make package command handle directories in extra classpath #3096

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

joan38
Copy link
Contributor

@joan38 joan38 commented Aug 14, 2024

Fixes #3095

@joan38 joan38 force-pushed the package-classpath branch 2 times, most recently from 7eaaa4f to a45ce34 Compare August 14, 2024 03:32
@Gedochao
Copy link
Contributor

@joan38 Hey, can you add an integration test for it?
Somewhere here would work:
https://github.com/VirtusLab/scala-cli/blob/main/modules/integration/src/test/scala/scala/cli/integration/PackageTestDefinitions.scala

@joan38 joan38 force-pushed the package-classpath branch 4 times, most recently from c0487f4 to 43d5c7f Compare August 15, 2024 15:52
@joan38
Copy link
Contributor Author

joan38 commented Aug 16, 2024

@Gedochao I was waiting for you to point where the tests are 😃 Thanks!
In the end the first PR version was completely wrong.
Now it's ready for review. Let me know

Copy link
Contributor

@Gedochao Gedochao left a comment

Choose a reason for hiding this comment

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

LGTM

@Gedochao Gedochao merged commit 6080642 into VirtusLab:main Aug 19, 2024
78 checks passed
@joan38 joan38 deleted the package-classpath branch August 22, 2024 17:59
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.

--classpath fails when it contains a directory in command package but works with compile
2 participants