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

Java 17 #356

Merged
merged 2 commits into from
Mar 22, 2024
Merged

Java 17 #356

merged 2 commits into from
Mar 22, 2024

Conversation

asolntsev
Copy link
Contributor

@asolntsev asolntsev commented Mar 22, 2024

Now Replay will require at least Java 17.
Before this change, Replay worked on Java 11.

@asolntsev asolntsev self-assigned this Mar 22, 2024
@asolntsev asolntsev added this to the 2.4.0 milestone Mar 22, 2024
@asolntsev asolntsev added dependencies java Pull requests that update Java code labels Mar 22, 2024
@asolntsev asolntsev merged commit 22ad3d1 into main Mar 22, 2024
3 checks passed
@asolntsev asolntsev deleted the java-17 branch March 22, 2024 15:38
@xabolcs
Copy link
Collaborator

xabolcs commented Mar 22, 2024

What's the value in requiring Java 17? Was a blocker for something big rewrite / addition?

@asolntsev
Copy link
Contributor Author

@xabolcs Yes, some of libraries used by Replay already require Java 17 for a while:

  • ECJ (Eclipse java compiler)
  • FlyingSaucer (PDF generation)
  • Selenide (UI tests)

E.g. I could not merge pull requests like this for a while: #354

xabolcs added a commit to xabolcs/re-play1 that referenced this pull request Apr 17, 2024
As stated in replay-framework#356, the following libraries require Java 17:
* ECJ (Eclipse java compiler, fastergt - the default template enginge)
* FlyingSaucer (PDF generation)
* Selenide (UI tests)

The helloworld-kotlin test project had to be manually adjusted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies java Pull requests that update Java code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants