-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
Server port not changing despite change in config/watch.yml #570
Labels
Comments
My Also related: luckyframework/lucky_cli#324. TL;DR: the process runners will inject their own PORT env var which overrides this. |
jwoertink
added a commit
that referenced
this issue
Mar 14, 2020
… managers don't override what we use. Fixes #570
5 tasks
djuber
pushed a commit
to djuber/lucky
that referenced
this issue
Aug 29, 2021
…le the signal and finish - rather than us immediately exiting and leaving it to log shutting down messages from the background (luckyframework#572) * lucky dev: When Ctrl+C is pressed, wait for the child process to handle the signal and finish - rather than us immediately exiting and leaving it to log shutting down messages from the background Resolves luckyframework#570. I've just used the approach I suggested there. I wasn't sure how/if to write a spec for this, so I haven't. Is that acceptable? To test it manually, I ran this from my app directory before and after the change to verify the behaviour: ```bash $ crystal run ~/Projects/lucky_cli/src/lucky.cr -- dev ``` * Update src/lucky_cli/process_runner.cr * oops. formatted Co-authored-by: Jeremy Woertink <jeremywoertink@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When I try to change my
config/watch.yml
to a different port, BrowserSync still proxy's to port 5000. It also ignores when I set a port for the environment like so:yarn run v1.9.4 $ path/to/app/node_modules/.bin/browser-sync start -c bs-config.js --port 3001 -p http://0.0.0.0:5000 web | [Browsersync] Proxying: http://0.0.0.0:5000
The text was updated successfully, but these errors were encountered: