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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

optionally skip npm install for web dependencies #34

Closed
osdevisnot opened this issue Apr 6, 2019 · 3 comments
Closed

optionally skip npm install for web dependencies #34

osdevisnot opened this issue Apr 6, 2019 · 3 comments

Comments

@osdevisnot
Copy link

First off, I want to give you guys props for this awesome project. Great Work 馃帀 馃帀

The more I use @pika-web, I feel like I should be able to skip NPM install step altogether.

Why would you want to do that?

This is why:
1_liNzl2MQKqg4tLMCF4jY5g

I am meaning to write full RFC on this, but wanted to throw quick thought to get some feedback.

Oh BTW, I did some exploration on this in #33

@FredKSchott
Copy link
Owner

So if I understand you correctly, the problem that this solves is: "npm install results in a huge node_modules folder, which is 馃憥"

I'm not sure if that's true, or at least it doesn't have to be. Frontend dependencies usually only end up making up a small part of your node_modules tree. And if you have any dev deps, they you're already npm installing anyway.

@osdevisnot
Copy link
Author

You are correct on both counts, understanding the original issue and the way you are thinking about node_modules being small for front end dependencies.

I guess I am wanting to solve huge npm deps problem without realizing most of them are actually devDevependencies.

Anyways, thanks for taking time to provide your feedback.

@FredKSchott
Copy link
Owner

np, thanks for asking the question!

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

No branches or pull requests

2 participants