Chore: Updates to modern npm and more...#365
Conversation
|
There was a problem hiding this comment.
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.
yarntonpmsimpleversion