Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Load CLI dependencies conditionally #173

Closed
Rich-Harris opened this issue Mar 6, 2018 · 1 comment
Closed

Load CLI dependencies conditionally #173

Rich-Harris opened this issue Mar 6, 2018 · 1 comment

Comments

@Rich-Harris
Copy link
Member

Running sapper --help requires express, cheerio, and other stuff that is only needed for certain tasks. We could speed up startup in many cases by lazily requiring it.

Ideally this would happen with dynamic imports — this would presumably involve changing the build setup and using Rollup's newer capabilities.

Rich-Harris added a commit that referenced this issue Mar 11, 2018
@Rich-Harris
Copy link
Member Author

Startup is now way faster, building Sapper itself is way faster, and we can now take advantage of TypeScript across the entire codebase (without losing type information when we import core.js).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant