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

bump puppeteer from 1.9.0 to 1.11.0 #5119

Merged
merged 1 commit into from
Jan 4, 2019
Merged

bump puppeteer from 1.9.0 to 1.11.0 #5119

merged 1 commit into from
Jan 4, 2019

Conversation

jivanvl
Copy link

@jivanvl jivanvl commented Dec 28, 2018

Issue:
To be created, but I'll explain here

At GitLab we're having an issue with puppeteer on our CI environment: https://gitlab.com/gitlab-org/csslab/-/jobs/139386734. Puppeteer fails to run the headless instance, updating puppeteer to 1.11.0 and testing locally with a docker image seem to solve the issue.

What I did

I updated the package.json file from addon-storyshots to point to the latest puppeteer version.

@storybook-safe-bot
Copy link
Contributor

Fails
🚫

PR is not labeled with one of: ["cleanup","doc-dependencies:update","BREAKING CHANGE","feature request","bug","documentation","maintenance","dependencies:update","dependencies","other"]

Generated by 🚫 dangerJS against c9275d1

@ndelangen ndelangen merged commit a442dba into storybookjs:next Jan 4, 2019
@jivanvl
Copy link
Author

jivanvl commented Jan 4, 2019

Thanks!

@ndelangen ndelangen added the patch:yes Bugfix & documentation PR that need to be picked to main branch label Jan 5, 2019
shilman pushed a commit that referenced this pull request Jan 6, 2019
@shilman shilman added the patch:done Patch/release PRs already cherry-picked to main/release branch label Jan 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants