Skip to content

Update README.md #724

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

Merged
merged 3 commits into from
Apr 16, 2018
Merged

Update README.md #724

merged 3 commits into from
Apr 16, 2018

Conversation

MacCracken
Copy link
Contributor

just incase users are using docker-compose version 3; delineated between version 2 and 3

just incase users are using docker-compose version 3; delineated between version 2 and 3
@ddavison
Copy link
Member

this is a nice doc update! have you made sure that this docker compose works?

@ddavison
Copy link
Member

ddavison commented Apr 16, 2018

I've answered my own question - it "does" work.

one more requested change, @MacCracken - could you add the yaml highlighting to the code block you've specified? Right now it will only render as preformatted text with no highlighting

ref: https://github.com/MacCracken/docker-selenium/blob/4872c71e9c8acd6c35b3612a02db639ac7a53255/README.md#version-3

@diemol
Copy link
Member

diemol commented Apr 16, 2018

@MacCracken please reference the variables HUB_HOST and HUB_PORT instead of HUB_PORT_4444_TCP_ADDR and HUB_PORT_4444_TCP_PORT

I've updated to comments to read for yaml, changing variables HUB_HOST and HUB_PORT haven't been tested by me on my side.  I can confirm these settings work.
updated the variables - had to test they didn't break the dockers
@diemol
Copy link
Member

diemol commented Apr 16, 2018

Looks good, thanks @MacCracken!

@diemol diemol merged commit aae1601 into SeleniumHQ:master Apr 16, 2018
@MacCracken MacCracken deleted the patch-7 branch April 16, 2018 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants