Skip to content
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

feat: use npm for managing js packages #571

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat: use npm for managing js packages #571

wants to merge 1 commit into from

Conversation

G-Rath
Copy link
Contributor

@G-Rath G-Rath commented Nov 21, 2024

We've decided that we want to use npm as our package manager for JavaScript dependencies, so this removes our explicit support for the other package managers, though we still remain pretty agnostic due to the use of PackageJson which I've retained for now given it's lightweight and secretly I'm keen to explore pnpm at some point so this will come in handy if that turns out to be worth switching to 😄

I've also retained the "create fake package managers" script + ci step to ensure that we really are only ever using npm; I plan to explore refactoring that script given we no longer need to support all the package managers as a follow-up since it too is lightweight and perfectly functional as-is.

@G-Rath
Copy link
Contributor Author

G-Rath commented Nov 21, 2024

(I've made this a draft for now because I decided I want to migrate a few more apps to ensure everything is smooth before we make it our default out-of-the-box setup)

@G-Rath G-Rath force-pushed the use-npm branch 3 times, most recently from adce0cd to c9054fa Compare November 22, 2024 03:24
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.

1 participant