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

Update Vagrantfile #2236

Merged
merged 1 commit into from Aug 26, 2016
Merged

Update Vagrantfile #2236

merged 1 commit into from Aug 26, 2016

Conversation

TylerJewell
Copy link

What does this PR do?

Fixes errors in the vagrant file related to switching to new docker launcher. Found issue with Che CLI using tty output when it's not available in a vagrant provisioner.

Fixed errors on the switch over to new docker launcher.
Improved readability of the output.
@TylerJewell TylerJewell merged commit 21704a6 into master Aug 26, 2016
@TylerJewell TylerJewell added this to the 4.7.0 milestone Aug 26, 2016
@TylerJewell TylerJewell added the kind/bug Outline of a bug - must adhere to the bug report template. label Aug 26, 2016
@TylerJewell TylerJewell self-assigned this Aug 26, 2016
@bmicklea bmicklea mentioned this pull request Aug 29, 2016
89 tasks
@TylerJewell TylerJewell deleted the vagrant-fixup branch September 1, 2016 13:23
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
Fixed errors on the switch over to new docker launcher.
Improved readability of the output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant