Skip to content

test commit - #29

Closed
ayushkalani wants to merge 1 commit into
sendgrid:masterfrom
ayushkalani:master
Closed

test commit#29
ayushkalani wants to merge 1 commit into
sendgrid:masterfrom
ayushkalani:master

Conversation

@ayushkalani

@ayushkalani ayushkalani commented Oct 10, 2017

Copy link
Copy Markdown

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

SendGridDX commented Oct 10, 2017

Copy link
Copy Markdown

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
Copy Markdown
Author

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

@mbernier

Copy link
Copy Markdown
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
Copy Markdown
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