Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

Conversation

@Brantone
Copy link

@Brantone Brantone commented May 2, 2019

No description provided.

@Brantone Brantone requested a review from zirkov May 2, 2019 17:04
@Brantone
Copy link
Author

Brantone commented May 2, 2019

Don't merge yet ... I'm still waiting on tests to complete ... just getting the PR in now so can get some eyes on it.

@zirkov
Copy link

zirkov commented May 2, 2019

LGTM!
By the way, Linux build nodes (docker based ones) are using AdoptOpenJdk container as their base (https://github.com/JetBrains/teamcity-docker-minimal-agent/blob/95ae4e7c836ce3199be126edb286f6b1f0e9e15e/ubuntu/Dockerfile#L1), so might make sense to force our Mac nodes to use it by default

@Brantone
Copy link
Author

Brantone commented May 2, 2019

Yeah, subsequent downstream cookbooks will also make use of that.

@Brantone Brantone merged commit 3ff83f8 into Roblox:master May 6, 2019
@Brantone
Copy link
Author

Brantone commented May 6, 2019

Need to keep eye on
sous-chefs#547
sous-chefs#544

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants