Skip to content

Conversation

@ryan-roemer
Copy link
Member

  • Switch from yarn to npm
  • Remove all benchmarks
  • Hone fixture down to just npm simple version

@changeset-bot
Copy link

changeset-bot bot commented Sep 3, 2025

⚠️ No Changeset found

Latest commit: be1e1e5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ryan-roemer ryan-roemer requested a review from Copilot September 3, 2025 17:43
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the project's tooling and testing infrastructure by switching from Yarn to npm and removing benchmarking functionality. The changes are part of a broader effort to simplify the project's development and CI/CD processes.

  • Removes all benchmark-related functionality including the test script, benchmark tests, and CI workflows
  • Switches from Yarn to npm for package management throughout the project
  • Streamlines the test structure to focus only on the npm "simple" scenario

Reviewed Changes

Copilot reviewed 27 out of 39 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test/script.js Completely removes the comprehensive test script that handled benchmarks and multiple package managers
test/packages/simple/yarn/* Removes Yarn-specific simple test scenario files
test/packages/complex/yarn/* Removes Yarn-specific complex test scenario files
test/packages/complex/npm/* Removes npm-specific complex test scenario files
test/packages/simple/* Updates plugin path and dependency versions in the remaining simple scenario
test/cli/package.spec.js Removes complex scenario tests, keeping only simple scenario
test/benchmark.js Completely removes benchmark validation tests
package.json Updates scripts to use npm, removes benchmark commands, updates author
README.md Removes benchmark sections and updates branding
.github/workflows/* Updates CI to use npm and removes benchmark steps

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ryan-roemer ryan-roemer merged commit a70cd2d into master Sep 3, 2025
4 checks passed
@ryan-roemer ryan-roemer deleted the chore/npm branch September 3, 2025 17:56
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