-
Notifications
You must be signed in to change notification settings - Fork 14
Chore: Updates to modern npm and more... #365
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
Conversation
|
There was a problem hiding this 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.
yarntonpmsimpleversion