Skip to content

(webdriverio): make puppeteer a peer dependency#12103

Merged
christian-bromann merged 3 commits intov9from
cb/puppeteer-as-peerDep
Jan 27, 2024
Merged

(webdriverio): make puppeteer a peer dependency#12103
christian-bromann merged 3 commits intov9from
cb/puppeteer-as-peerDep

Conversation

@christian-bromann
Copy link
Member

Proposed changes

The webdriverio package should not depend on Puppeteer. If the user doesn't use the getPuppeteer command there is no need for the dependency to be installed.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Further comments

fixes #10726

Reviewers: @webdriverio/project-committers

@christian-bromann christian-bromann added PR: Breaking Change 💥 PRs that contain breaking changes v9 labels Jan 26, 2024
@christian-bromann christian-bromann added this to the v9 milestone Jan 26, 2024
@christian-bromann christian-bromann merged commit 3d5670f into v9 Jan 27, 2024
@christian-bromann christian-bromann deleted the cb/puppeteer-as-peerDep branch January 27, 2024 03:19
@christian-bromann christian-bromann restored the cb/puppeteer-as-peerDep branch January 28, 2024 18:23
christian-bromann added a commit that referenced this pull request Feb 2, 2024
* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 remove jsonWireProtocol

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* (internal): update some dependencies that couldn't be updated due to support for Node.js v16

closes #11938
fixes #11858

* (webdriverio): make puppeteer a peer dependency (#12103)

* (webdriverio): make puppeteer a peer dependency

* fix type issues

* fix unit tests

---------

Co-authored-by: Erwin Heitzman <15839059+erwinheitzman@users.noreply.github.com>
Co-authored-by: Christian Bromann <git@bromann.dev>
christian-bromann added a commit that referenced this pull request Feb 17, 2024
* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 remove jsonWireProtocol

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* (internal): update some dependencies that couldn't be updated due to support for Node.js v16

closes #11938
fixes #11858

* (webdriverio): make puppeteer a peer dependency (#12103)

* (webdriverio): make puppeteer a peer dependency

* fix type issues

* fix unit tests

---------

Co-authored-by: Erwin Heitzman <15839059+erwinheitzman@users.noreply.github.com>
Co-authored-by: Christian Bromann <git@bromann.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Breaking Change 💥 PRs that contain breaking changes v9

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant