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

Add support for IPC options on docker run #148

Merged
merged 1 commit into from
Jan 10, 2020

Conversation

underscorebrody
Copy link

No description provided.

@fforres
Copy link

fforres commented Nov 21, 2019

Do you folks have any plans on merging this?
We where looking at the cypress docs mentioning this being a source of flakiness.
https://docs.cypress.io/guides/references/error-messages.html#The-Chromium-Renderer-process-just-crashed

Anyhting we can do to help test this?

ping @lox ? :D

@jlison
Copy link

jlison commented Nov 27, 2019

I agree with fforres. It will be great if we could get this merged as soon as possible.

@deluxetom
Copy link

This is a great PR, super useful in our situation, can this be merged?

@fforres
Copy link

fforres commented Dec 2, 2019

CC @lox || @toolmantim 😄

@jlison
Copy link

jlison commented Dec 10, 2019

Bump! 😅

@toolmantim
Copy link
Contributor

Thanks @underscorebrody for the contribution, and sorry for the delay! Your changes look great… thanks for updating the readme and tests too ❤️

@toolmantim toolmantim merged commit c3acef0 into buildkite-plugins:master Jan 10, 2020
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.

None yet

7 participants