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

Syncrhonize chrome and chrome-driver versions #55

Merged
merged 1 commit into from
Nov 24, 2020

Conversation

tramuntanal
Copy link
Contributor

@tramuntanal tramuntanal commented Nov 19, 2020

Chrome and chrome driver versions have diverged.
This PR synchronizes both in the Dockerfile-test.

@tramuntanal tramuntanal self-assigned this Nov 19, 2020
Copy link
Contributor

@oliverbarnes oliverbarnes left a comment

Choose a reason for hiding this comment

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

Just a note about the bundler line

@@ -17,7 +17,7 @@ RUN curl -sL https://deb.nodesource.com/setup_10.x | bash - \

RUN npm install -g [email protected]

RUN gem install bundler \
RUN gem install bundler --version '>= 2.1.4' \
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed if it'll build with ruby > 2.6, which comes with bundler built in?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added it because I was getting an error on the bundler version because it was something like 1.17...

Copy link
Contributor

Choose a reason for hiding this comment

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

Ahh, I see - it comes with an older version :/ I double checked now and it is indeed at 1.17.2:

➜  decidim-docker git:(sandbox) docker run --name ruby -i -t ruby:2.6.6 bash
root@2ce5d8d66e46:/# ruby -v
ruby 2.6.6p146 (2020-03-31 revision 67876) [x86_64-linux]
root@2ce5d8d66e46:/# bundle -v
Bundler version 1.17.2

Copy link

@Leusev Leusev left a comment

Choose a reason for hiding this comment

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

Ok, looks good for me @tramuntanal 👍

Copy link

@ivan-mr ivan-mr left a comment

Choose a reason for hiding this comment

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

Thanks @tramuntanal

Copy link
Contributor

@oliverbarnes oliverbarnes left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me too

@tramuntanal tramuntanal merged commit 594a1db into master Nov 24, 2020
@tramuntanal tramuntanal deleted the chore/synchronize_chrome_and_chrome_driver branch November 24, 2020 07:50
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.

4 participants