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

Remove JDK 7 from build documentation. #6031

Merged
merged 1 commit into from
Jul 27, 2018
Merged

Remove JDK 7 from build documentation. #6031

merged 1 commit into from
Jul 27, 2018

Conversation

yurmix
Copy link
Contributor

@yurmix yurmix commented Jul 20, 2018 •
edited by leventov
Loading

Fixes #6030

@yurmix yurmix mentioned this pull request Jul 25, 2018
Closed
@jihoonson
Copy link
Contributor

Ah, great! Would you fill out our CLA here?

Also, if you can change the PR comment to be start with Fixes #6030, the issue #6030 is automatically closed when this PR is merged. See https://help.github.com/articles/closing-issues-using-keywords/ for more details.

@yurmix
Copy link
Contributor Author

yurmix commented Jul 25, 2018

@jihoonson Now that the project is under Apache, shouldn't it be Apache ICLA?
If so, I also believe this CLA link should be updated (possibly in a separate PR 😄).

@yurmix
Copy link
Contributor Author

yurmix commented Jul 26, 2018 •
edited
Loading

@jihoonson Thank you for helping with my first pr!
I have submitted one to Apache and it should be available to Druid PMCs. You can see my name (Eyal Yurman) showing here: CLAs without ids

@jihoonson
Copy link
Contributor

@yurmix thanks!

@jihoonson jihoonson merged commit 94d6c9a into apache:master Jul 27, 2018
@dclim dclim added this to the 0.13.0 milestone Oct 8, 2018
@terrorizer1980 terrorizer1980 mentioned this pull request Oct 27, 2023
Closed
@ajesse11x ajesse11x mentioned this pull request Oct 27, 2023
Open
@soltan soltan mentioned this pull request Oct 27, 2023
Open
@soltan soltan mentioned this pull request Dec 27, 2023
Open
@terrorizer1980 terrorizer1980 mentioned this pull request Dec 27, 2023
Closed
@ajesse11x ajesse11x mentioned this pull request Dec 28, 2023
Open
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
0.13.0
Development

Successfully merging this pull request may close these issues.
3 participants
@yurmix @jihoonson @dclim