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
Allow creating dev server without opening port. #2314
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/pikapkg/snowpack/91vqbyo3q |
The failed unit test is caused by another bug. |
Test failures are fixed in this PR #2328 |
Added a config for `devOptions.server?: boolean` which defaults to `true`; If `false`, then no http(s) server will be created and it will be up to the caller to create the initial server. Created a stub for EsmHmrEngine that avoids any processing if HMR is disabled.
rebased to keep up with @FredKSchott speedy fixes |
Fixed in #2707 by passing port = 0 (will add some docs for this). In the next breaking change for Snowpack, I'd like to add a |
Added a config for
devOptions.server?: boolean
which defaults totrue
;If
false
, then no http(s) server will be created and it will be up tothe caller to create the initial server.
Created a stub for EsmHmrEngine that avoids any processing if HMR is disabled.
Changes
Testing
TODO: Add Tests
Docs
TODO: Update Docs