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

Support coursier-downloaded scala wrapper tests on Windows #3325

Conversation

Gedochao
Copy link
Contributor

@Gedochao Gedochao commented Nov 28, 2024

This allows us to run tests with the scala runner wrapper coming from a coursier installation of the language.
We swap out the Scala CLI launcher used in the wrapper for one built for the purpose of our integration tests.
We used to skip running this on Windows until this change.

@Gedochao Gedochao changed the title [WIP] Support coursier scala launcher tests on Windows [WIP] Support coursier-downloaded scala wrapper tests on Windows Nov 28, 2024
@Gedochao Gedochao force-pushed the maintenance/coursier-installation-tests-on-windows branch 4 times, most recently from dca8274 to 9ec5d3d Compare November 29, 2024 09:52
@Gedochao Gedochao changed the title [WIP] Support coursier-downloaded scala wrapper tests on Windows Support coursier-downloaded scala wrapper tests on Windows Nov 29, 2024
@Gedochao Gedochao requested a review from tgodzik November 29, 2024 09:54
@Gedochao
Copy link
Contributor Author

Note: this (hopefully) might make it easier to reproduce this issue:

@Gedochao Gedochao marked this pull request as ready for review November 29, 2024 09:55
@Gedochao Gedochao force-pushed the maintenance/coursier-installation-tests-on-windows branch 2 times, most recently from f2dc4a0 to 4cf48ae Compare November 29, 2024 13:54
@Gedochao Gedochao force-pushed the maintenance/coursier-installation-tests-on-windows branch from 4cf48ae to e338513 Compare December 2, 2024 08:20
@Gedochao Gedochao requested review from kasiaMarek and tgodzik and removed request for tgodzik December 3, 2024 10:22
@Gedochao Gedochao merged commit 2f0c7b8 into VirtusLab:main Dec 4, 2024
78 checks passed
@Gedochao Gedochao deleted the maintenance/coursier-installation-tests-on-windows branch December 4, 2024 11:37
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.

2 participants