-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat(config-cli): new command tool to bundle config json from a config path #2252
Conversation
packages/config-cli/README.md
Outdated
Bundle config files into config bundle json from a given config path. | ||
|
||
```bash | ||
./bin/bmc.js screenshot --config config/ --bundle config/config.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bundle
?
bundling into the output directory could be interesting, maybe make the recommendation just --output config.json
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
host = `http://localhost:${port}`; | ||
BasemapsServer = await startServer(host, port, config, logger); | ||
// Force a default url base so WMTS requests know their relative url | ||
process.env[Env.PublicUrlBase] = host; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this is set we need to respect it we should not be overwriting env vars.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can set this back after the script running in finally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proab safer to just use it if it exists the same as basemaps-server cli
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, updated.
No description provided.