search

sveltejs / svelte-cli

Command line interface for Svelte
MIT License
104 stars 18 forks source link

svelte is both a dev dep and runtime dep... ? #18

Closed elidoran closed 7 years ago

elidoran commented 7 years ago

dev dep runtime dep

Should it only be a runtime dep?

Rich-Harris commented 7 years ago

It's a dev dep for the purposes of testing... but yeah, it doesn't make a huge amount of sense. I suspect what happened is we needed a more up-to-date version to get some tests to pass, and I did npm install -D svelte@latest instead of npm install -S svelte@latest.

They'll both be satisfied by the highest version below 2 though, so it shouldn't result in duplicate copies.

elidoran commented 7 years ago

I thought so. Figured I'd mention it in case it was escaping notice because I've done it a few times.

Extra babble: I learned npm update -S (or -D) is handy to avoid that. Won't add it like install. Only updates what's listed.

Thank you for the quick reply. :)