Skip to content
This repository was archived by the owner on Dec 21, 2024. It is now read-only.

Reworking java installation guide to avoid leading the user to use the java8 version which will not be supported anymore #767

Closed
BreezeBlocksTool opened this issue Jun 28, 2022 · 0 comments · Fixed by #783

Comments

@BreezeBlocksTool
Copy link

👓 What did you see?

The documentation on java installation with Maven gives the dependency for the java8 version first https://cucumber.io/docs/installation/java/ whereas this version will not be upgraded to work with java 12+. I think it should either be removed, or at least it should be documented that this version has an expiration date...

✅ What did you expect to see?

When I first installed cucumber a few months ago, I was looking for arguments to choose between the java or java8 version. The only one I could find back then was that using the java8 version avoids us the pain of finding a method name that's almost always a duplicate from the step.
A few months later I found out that there was no easy way with the java8 version to find all uses of a step from the glue code. Finding the glue code from the feature file is easy in intellij, but the other way around is not. The version using annotations handles that better, so that's an argument I would have liked to see in the documentation to make an informed choice.
And today I found out that the java8 version will not support java12+ due to technical difficulties... That's a game changer and I had to adapt a lot of code to migrate. I think this should be written in capital letters, red and flashing in the documentation, to save future developers like me from the pain of the migration...

📦 Which tool/library version are you using?

Java 17 + Maven

📚 Any additional context?

see issue for more context: https://github.com/cucumber/cucumber-jvm/issues/1817

mpkorstanje added a commit that referenced this issue Aug 11, 2022

Verified

This commit was signed with the committer’s verified signature.
mpkorstanje M.P. Korstanje
Cucumber Java8 is considered for deprecation[1]. Recommending it in the docs
suggests otherwise.

1: cucumber/cucumber-jvm#2279

Fixes: #767
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant