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

Dockerfile: Pin ruby and Go version #208

Merged
merged 3 commits into from Jul 30, 2017
Merged

Dockerfile: Pin ruby and Go version #208

merged 3 commits into from Jul 30, 2017

Conversation

@jayvdb
Copy link
Member

@jayvdb jayvdb commented Jul 14, 2017

No description provided.

Closes #213
@gitmate-bot
Copy link

@gitmate-bot gitmate-bot commented Jul 22, 2017

Hey! This pull request hasn't been updated for a while :/ It would be nice if we could get this going again!

1 similar comment
@gitmate-bot
Copy link

@gitmate-bot gitmate-bot commented Jul 22, 2017

Hey! This pull request hasn't been updated for a while :/ It would be nice if we could get this going again!

@jayvdb jayvdb force-pushed the jayvdb:ruby2.2 branch 3 times, most recently from 1e02d5f to 99e5f3d Jul 22, 2017
jayvdb added 2 commits Jul 14, 2017
Release 0.11 used Ruby 2.2.
These shouldnt change to newer versions without
an explicit upgrade.

Related to #144
Fixes #216
Release 0.11 used Go 1.7.
These shouldnt change to newer versions without
an explicit upgrade.

Related to #144
@yash-nisar
Copy link
Member

@yash-nisar yash-nisar commented Jul 30, 2017

@jayvdb
Copy link
Member Author

@jayvdb jayvdb commented Jul 30, 2017

@jayvdb
Copy link
Member Author

@jayvdb jayvdb commented Jul 30, 2017

@rultor merge

@rultor
Copy link

@rultor rultor commented Jul 30, 2017

@rultor merge

@jayvdb OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 555be8d into coala:master Jul 30, 2017
5 checks passed
5 checks passed
ci/dockercloud Your tests passed in Docker Cloud
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
review/gitmate/commit This commit has no issues. :)
Details
review/gitmate/manual This commit was acknowledged.
Details
review/gitmate/pr This PR has no issues. :)
Details
@rultor
Copy link

@rultor rultor commented Jul 30, 2017

@rultor merge

@jayvdb Done! FYI, the full log is here (took me 2min)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.