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

Fix the Java 8 CI #14655

Merged
merged 1 commit into from
Mar 9, 2022
Merged

Fix the Java 8 CI #14655

merged 1 commit into from
Mar 9, 2022

Conversation

smarter
Copy link
Member

@smarter smarter commented Mar 9, 2022

This disables the test added in #14643 which broke the CI because it requires
Java 9+ and unfortunately we don't have a way to selectively run tests on
specific Java versions currently.

[test_java8]

This disables the test added in scala#14643 which broke the CI because it requires
Java 9+ and unfortunately we don't have a way to selectively run tests on
specific Java versions currently.
@smarter
Copy link
Member Author

smarter commented Mar 9, 2022

Merging now to unbreak the CI.

@smarter smarter merged commit e078e79 into scala:main Mar 9, 2022
@smarter smarter deleted the fix-java8-ci branch March 9, 2022 18:32
@som-snytt
Copy link
Contributor

For my Coursera in vulpix I have a class assignment to forward port the javaVersion test comment from last year. Lukas just reminded me to please use it in a test, a great simplification.

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.

3 participants