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: upgrade puppeteer to 21.0.1 #15331

Merged
merged 4 commits into from
Aug 3, 2023
Merged

deps: upgrade puppeteer to 21.0.1 #15331

merged 4 commits into from
Aug 3, 2023

Conversation

adamraine
Copy link
Member

@adamraine adamraine requested a review from a team as a code owner August 3, 2023 00:13
@adamraine adamraine requested review from connorjclark and removed request for a team August 3, 2023 00:13

// Puppeteer uses a later version of rxjs as a dev dep.
// We don't need it for normal execution, but these types are necessary
// when we reach into puppeteer internals in `lightrider-entry.js`
Copy link
Member Author

Choose a reason for hiding this comment

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

Could install rxjs as a dev dep instead, but this gives us an opportunity to explain ourselves in a comment in case this isn't necessary in the future.

core/gather/driver.js Outdated Show resolved Hide resolved
@adamraine adamraine changed the title deps: upgrade puppeteer to 21.0.0 deps: upgrade puppeteer to 21.0.1 Aug 3, 2023
@adamraine adamraine merged commit 101b186 into main Aug 3, 2023
@adamraine adamraine deleted the pptr-up-22 branch August 3, 2023 18:38
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.

2 participants