Skip to content

test commit #29

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

Closed
wants to merge 1 commit into from
Closed

test commit #29

wants to merge 1 commit into from

Conversation

ayushkalani
Copy link

@ayushkalani ayushkalani commented Oct 10, 2017

Test for this issue TravisCI failing with bad JAVA_HOME env var #28

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Oct 10, 2017
@SendGridDX
Copy link

SendGridDX commented Oct 10, 2017

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ayushkalani
Copy link
Author

We need to check this out
travis-ci/travis-ci#7019
travis-ci/travis-ci#7884

@mbernier
Copy link
Contributor

@thinkingserious according to the second link from @ayushkalani, it looks like Travis has marked this as a wontfix for oraclejdk7 because Oracle has decided not to support this build see this comment

Should we just remove the oraclejdk7 dependency in travis.yml?

@thinkingserious
Copy link
Contributor

@mbernier,

Yes, and merged: #35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: code review request requesting a community code review or review from Twilio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants