Skip to content

(@wdio/cli): implement constistent and simple package manager detection#11764

Merged
erwinheitzman merged 5 commits intowebdriverio:mainfrom
erwinheitzman:implement-consistent-and-simple-pm-detection
Dec 1, 2023
Merged

(@wdio/cli): implement constistent and simple package manager detection#11764
erwinheitzman merged 5 commits intowebdriverio:mainfrom
erwinheitzman:implement-consistent-and-simple-pm-detection

Conversation

@erwinheitzman
Copy link
Member

Proposed changes

Attempt to fix the last remaining issues mentioned in #11581

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)

Reviewers: @webdriverio/project-committers

Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

Can we have a unit test for detectPackageManager?

Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

Minor tweak suggestion, once merged this is good to go 👍

@erwinheitzman erwinheitzman merged commit a0c3201 into webdriverio:main Dec 1, 2023
@erwinheitzman erwinheitzman deleted the implement-consistent-and-simple-pm-detection branch December 1, 2023 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Bug Fix 🐛 PRs that contain bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants