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

Fix Chrome timeout in Travis builds #10

Merged
merged 6 commits into from Jul 25, 2019
Merged

Fix Chrome timeout in Travis builds #10

merged 6 commits into from Jul 25, 2019

Conversation

@creazero
Copy link
Contributor

@creazero creazero commented Jul 24, 2019

Turns out, Travis image has its own Chrome, which doesn't want to start for some reason. So I added a version guard for it (>= 75) and additional env vars from puppeteer's install.js.

Fixes #9
@ai

scripts/findChrome.js Outdated Show resolved Hide resolved
Loading
scripts/findChrome.js Outdated Show resolved Hide resolved
Loading
scripts/findChrome.js Show resolved Hide resolved
Loading
ai
ai approved these changes Jul 24, 2019
@ai
Copy link
Contributor

@ai ai commented Jul 24, 2019

@mbalabash LGTM. We can release it under patch (as downloader fix) or minor (since we added new env variables) release.

Loading

@mbalabash
Copy link
Owner

@mbalabash mbalabash commented Jul 25, 2019

Wow, awesome work!
Thanks you guys

Loading

@mbalabash mbalabash merged commit 7b46d13 into mbalabash:master Jul 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants