Skip to content

openjdk#790

Merged
AndKram merged 2 commits intomasterfrom
openjdk
Jan 25, 2019
Merged

openjdk#790
AndKram merged 2 commits intomasterfrom
openjdk

Conversation

@AndKram
Copy link
Copy Markdown
Collaborator

@AndKram AndKram commented Jan 22, 2019

Update docs to refer to http://openjdk.java.net/

@AndKram AndKram requested a review from DaveyBiggers January 22, 2019 15:29
@katja-hofmann katja-hofmann self-requested a review January 23, 2019 14:34
Copy link
Copy Markdown
Member

@katja-hofmann katja-hofmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The installation instructions for windows are not clear to me. Is JDK 8 needed or would 9 work? The link to openjdk is generic, is there a more direct link to the windows installation instructions for the required version?

@AndKram
Copy link
Copy Markdown
Collaborator Author

AndKram commented Jan 23, 2019

It has to be Java 8.

@AndKram AndKram merged commit 08cad2f into master Jan 25, 2019
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