Skip to content

Conversation

@kenrick95
Copy link
Contributor

Fixes lwsjs/local-web-server#115

but I'm not really sure if this is the best way to achieve it.

What this PR does?

  • Moved the reading stored config function to util file
  • Used that util file in both index.js and lib/command/serve.js
  • In index.js, use deepmerge to merge the hardcoded init options, the options from listen() argument, and the options from config file
  • Added test: test/lws.js

@75lb 75lb merged commit b5a490f into lwsjs:master May 4, 2019
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.

Programmatic API: configFile option is not honored

2 participants