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.

Sign up for GitHub

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

Jump to bottom

Use Java 17 in CI #26

Merged
merged 4 commits into from Dec 3, 2023
Merged

Use Java 17 in CI #26

merged 4 commits into from Dec 3, 2023

Conversation

basilevs
Copy link
Contributor

@basilevs basilevs commented Dec 2, 2023 •
edited

This is required by #21

@basilevs
Trying to fix NPE
919dc0c 
java.lang.NullPointerException
at org.eclipse.rcptt.reporting.html.FullSingleTestHtmlRenderer.renderEvent(FullSingleTestHtmlRenderer.java:230)
at org.eclipse.rcptt.reporting.html.FullSingleTestHtmlRenderer.renderSnapshots(FullSingleTestHtmlRenderer.java:169)
@basilevs
Fix invalid command line syntax (?) in tyho-surefire config
020f982
@basilevs
Copy link
Contributor Author

basilevs commented Dec 2, 2023 •
edited

I've updated Java to 17, but Tycho surefire ignored --add-opens argument and the tests were failing.
Had to remove line breaks from argument line configuration.

@basilevs basilevs merged commit 172b50e into master Dec 3, 2023
1 of 2 checks passed
@basilevs basilevs deleted the java17 branch December 3, 2023 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@basilevs