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

LUCENE-9865: Reduce unnecessary bla-bla-bla in top-level readme file #39

Merged
merged 1 commit into from
Mar 24, 2021

Conversation

dweiss
Copy link
Contributor

@dweiss dweiss commented Mar 24, 2021

No description provided.

@dweiss dweiss merged commit c23ea2f into apache:main Mar 24, 2021
@mocobeta
Copy link
Contributor

A bit off-topic...
Would it be better to link to jdk.java.net or AdoptOpenJDK instead of Oracle Java from the README?

@dweiss
Copy link
Contributor Author

dweiss commented Mar 25, 2021

I don't know - perhaps! Please go ahead and change it if you like, @mocobeta

@mocobeta
Copy link
Contributor

@dweiss Thanks, I'd like to open another follow-up PR. (I thought it'd be better that we drop the link to OracleJDK from our official docs; it now cannot be used for production for free anyway.)

gsmiller pushed a commit to gsmiller/lucene that referenced this pull request Mar 30, 2021
gsmiller pushed a commit to gsmiller/lucene that referenced this pull request Mar 30, 2021
gsmiller pushed a commit to gsmiller/lucene that referenced this pull request Mar 30, 2021
janhoy pushed a commit to cominvent/lucene that referenced this pull request May 12, 2021
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