voideditor / void

https://voideditor.com
MIT License
8.01k stars 396 forks source link

feat: use npm instead of yarn #30

Closed quant-eagle closed 1 month ago

quant-eagle commented 1 month ago

Summary

This PR transitions the project from Yarn to npm for package management. This change aligns with the decision by the VS Code maintainers to move away from Yarn. Npm has caught up in features, notably with automatic peer dependency installation (npm 7+), and Yarn v1 is no longer actively maintained.

Changes

Why is this needed?

Yarn v1 has become outdated, while npm now provides similar functionality and better long-term support. This transition aligns with the upstream VS Code repository to ease integration and reduce maintenance efforts.

Checklist

Testing

Ensure that all contributors are using npm, and that the project builds and tests run successfully with the new package manager.

Additional Information

Switching to npm helps keep this fork aligned with VS Code upstream, ensuring smoother dependency management and long-term stability. Should you face any issues, please report them for resolution.

andrewpareles commented 4 days ago

Any idea how to do nonlocal imports with npm @deepak1556 or @quant-eagle? If you could help me do a PR where void.contribution.ts simply imports 'openai' and prints it out, that'd be a ton of help right now.

I'm getting in touch because it looks like migrating from yarn to npm makes imports a lot more difficult. I spent a lot of time trying to import 'openai' in void.contribution.ts, and am considering reverting to yarn because I think it'd be easier. Let me know what you think.