Skip to content

Update CI Install Scripts#16

Merged
promiseofcake merged 2 commits intomasterfrom
ljk/bazel-point-release
May 26, 2018
Merged

Update CI Install Scripts#16
promiseofcake merged 2 commits intomasterfrom
ljk/bazel-point-release

Conversation

@promiseofcake
Copy link
Copy Markdown
Contributor

Various fixes for installing bazel in CI

Builds without an embedded JDK are no longer being built.

@promiseofcake promiseofcake requested a review from a team May 24, 2018 19:04
@promiseofcake promiseofcake changed the title Update CI Update CI Install Scripts May 25, 2018
@promiseofcake promiseofcake requested a review from vladlosev May 25, 2018 02:12
Copy link
Copy Markdown

@vladlosev vladlosev left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link
Copy Markdown

@melindalu melindalu left a comment

Choose a reason for hiding this comment

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

🙇 thank you for the fix, LGTM :shipit:

@promiseofcake promiseofcake merged commit 7074e07 into master May 26, 2018
@promiseofcake promiseofcake deleted the ljk/bazel-point-release branch May 26, 2018 06:08
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.

4 participants