Skip to content
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

[now dev] Add --listen / -l flag, deprecate --port / -p #2720

Merged
merged 12 commits into from Aug 9, 2019

Conversation

leo
Copy link
Contributor

@leo leo commented Aug 7, 2019

Deprecates --port in favor of a new --listen flag, which matches the behavior of serve. This allows for binding to a specific network interface, as well as Unix socket files or Windows named pipes.

TooTallNate and others added 2 commits August 5, 2019 13:33
This changes the default network interface that `now dev` binds to, in
order to prevent operating system firewalls from showing a confirmation
prompt in order to run.

Fixes #2704.
Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought @TooTallNate already made this change 🤔

@TooTallNate
Copy link
Member

@styfle It was reverted in 8638d17 because we are going to change it to --listen instead.

@TooTallNate TooTallNate changed the title [now dev] Add --bind / -b flag [now dev] Add --listen / -l flag Aug 8, 2019
@TooTallNate TooTallNate changed the title [now dev] Add --listen / -l flag [now dev] Add --listen / -l flag, deprecate --port / -p Aug 9, 2019
@codecov-io
Copy link

codecov-io commented Aug 9, 2019

Codecov Report

Merging #2720 into canary will increase coverage by 0.14%.
The diff coverage is 46.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           canary    #2720      +/-   ##
==========================================
+ Coverage   13.57%   13.71%   +0.14%     
==========================================
  Files         261      262       +1     
  Lines       10536    10569      +33     
  Branches     1356     1366      +10     
==========================================
+ Hits         1430     1450      +20     
- Misses       8961     8970       +9     
- Partials      145      149       +4
Impacted Files Coverage Δ
src/util/dev/types.ts 0% <ø> (ø) ⬆️
src/commands/dev/index.ts 0% <0%> (ø) ⬆️
src/commands/dev/dev.ts 0% <0%> (ø) ⬆️
src/util/dev/server.ts 61.16% <50%> (-0.13%) ⬇️
src/util/dev/parse-listen.ts 76% <76%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f48bb49...6253d7a. Read the comment docs.

@TooTallNate TooTallNate merged commit 38bc7cc into canary Aug 9, 2019
@TooTallNate TooTallNate deleted the update/now-dev-add-bind branch August 9, 2019 18:16
TooTallNate added a commit that referenced this pull request Aug 9, 2019
kodiakhq bot pushed a commit that referenced this pull request Aug 9, 2019
AndyBitz pushed a commit that referenced this pull request Aug 20, 2019
* [now dev] Add `--bind` / `-b` flag

This changes the default network interface that `now dev` binds to, in
order to prevent operating system firewalls from showing a confirmation
prompt in order to run.

Fixes #2704.

* Change to `--listen`

* Default port 3000

* Fix unit

* Fix `--port`

* Use `-l` for integration tests

* Add unit tests
AndyBitz pushed a commit that referenced this pull request Aug 20, 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.

None yet

5 participants