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

deps(puppeteer): upgrade to 16.1.0 #14244

Merged
merged 16 commits into from
Aug 8, 2022
Merged

deps(puppeteer): upgrade to 16.1.0 #14244

merged 16 commits into from
Aug 8, 2022

Conversation

adamraine
Copy link
Member

Split of from #13901

At some point along the way, pptr added a package.json for the esm modules. This would be nice to have if we want to use the esm versions of the modules.

This also fixes some outdated chrome issues with #14146

@adamraine adamraine requested a review from a team as a code owner July 28, 2022 00:03
@adamraine adamraine requested review from brendankenny and removed request for a team July 28, 2022 00:03
@connorjclark connorjclark changed the title deps(puppeteer): upgrade to 15.2.0 deps(puppeteer): upgrade to 15.5.0 Jul 29, 2022
@adamraine adamraine changed the title deps(puppeteer): upgrade to 15.5.0 deps(puppeteer): upgrade to 16.1.0 Aug 8, 2022
@adamraine
Copy link
Member Author

Bumping to 16:

https://github.com/puppeteer/puppeteer/releases/tag/v16.0.0

This is the release with changes to how puppeteer handles iframes, so I am going to run our DT / FR tests a few times to see if any flakiness reeappers.

@@ -216,7 +216,7 @@ describe('Lighthouse Viewer', () => {
document.querySelector('.lh-tools__dropdown')).visibility === 'visible';
});

// For some reason, clicking this button doesn't always initiate the download after upgrading to Puppeteer 15.2.0.
// For some reason, clicking this button doesn't always initiate the download after upgrading to Puppeteer 16.
Copy link
Collaborator

Choose a reason for hiding this comment

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

why update this line? presumably the version it broke in is useful information.

Copy link
Member Author

Choose a reason for hiding this comment

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

I didn't do a bisect to find the revision that broke it, this is just calling out the version upgrade that introduced the problem for us.

@adamraine adamraine merged commit cf55661 into master Aug 8, 2022
@adamraine adamraine deleted the pptr-15 branch August 8, 2022 18:12
alexnj pushed a commit to alexnj/lighthouse that referenced this pull request Aug 24, 2022
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.

4 participants