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.

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

Switch from pony-stable to corral #107

Merged
merged 1 commit into from
May 6, 2020
Merged

Switch from pony-stable to corral #107

merged 1 commit into from
May 6, 2020

Conversation

SeanTAllen
Copy link
Member

Pony-stable is deprecated. This is part of our move off of it.
Corral supports transitive dependencies so we can go back and
change the corral.json later when we add corral support to the
dependencies we have listed.

@SeanTAllen SeanTAllen added the do not merge This PR should not be merged at this time label May 6, 2020
@SeanTAllen SeanTAllen requested a review from Theodus May 6, 2020 03:11
@SeanTAllen SeanTAllen force-pushed the stable-to-corral branch 4 times, most recently from 79f82ad to 9cae344 Compare May 6, 2020 03:23
@SeanTAllen
Copy link
Member Author

@Theodus when you merge this, can you add a note to LWIP about how we are starting our dog-fooding of the transition from pony-stable to corral and perhaps a bit of background on that. Seems like a very good LWIP note.

@SeanTAllen SeanTAllen removed the do not merge This PR should not be merged at this time label May 6, 2020
@@ -1,8 +1,8 @@
FROM ponylang/ponyup-ci-x86-64-unknown-linux-builder:release AS build
Copy link
Member Author

Choose a reason for hiding this comment

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

i can't even find the Dockerfile for this. It's so old. And it never existed in this repo. Oooops.

@SeanTAllen SeanTAllen force-pushed the stable-to-corral branch 3 times, most recently from e7982f5 to 542eee3 Compare May 6, 2020 11:47
Pony-stable is deprecated. This is part of our move off of it.
Corral supports transitive dependencies so we can go back and
change the corral.json later when we add corral support to the
dependencies we have listed.

Closes #105
@Theodus
Copy link
Contributor

Theodus commented May 6, 2020

I am unable to reproduce the CI failure

Copy link
Contributor

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

LGTM

@Theodus Theodus merged commit b9bf730 into master May 6, 2020
@Theodus Theodus deleted the stable-to-corral branch May 6, 2020 13:58
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.

2 participants