Skip to content

Open localhost URL with --open#28

Merged
kylecarbs merged 2 commits into
coder:masterfrom
fand:cli-open
Mar 7, 2019
Merged

Open localhost URL with --open#28
kylecarbs merged 2 commits into
coder:masterfrom
fand:cli-open

Conversation

@fand

@fand fand commented Mar 6, 2019

Copy link
Copy Markdown
Contributor

CLI option --open is not working now.

I implemented --open with this npm package:
https://github.com/sindresorhus/opn

@fand fand closed this Mar 6, 2019
@fand fand reopened this Mar 6, 2019
Comment thread packages/server/src/cli.ts
Comment thread packages/server/src/cli.ts Outdated
Co-Authored-By: fand <fand@users.noreply.github.com>
@fand

fand commented Mar 7, 2019

Copy link
Copy Markdown
Contributor Author

seems tests are failing beacuse of API rate limit on cloning vscode-ripgrep.
I'll close/reopen PR just to trigger rebuild.

@fand fand closed this Mar 7, 2019
@fand fand reopened this Mar 7, 2019
@sr229

sr229 commented Mar 7, 2019

Copy link
Copy Markdown
Contributor

@frand It's a Travis-side issue. NPMJS seems to ratelimit the CI fsr, recommend you wait it out for 3 hours

@fand fand closed this Mar 7, 2019
@fand fand reopened this Mar 7, 2019
@kylecarbs

Copy link
Copy Markdown
Member

We'll be fixing the CI for PRs soon.

@kylecarbs kylecarbs merged commit 87d2e22 into coder:master Mar 7, 2019
code-asher pushed a commit that referenced this pull request Jun 19, 2019
* Open localhost URL with --open

* Catch errors on opn

Co-Authored-By: fand <fand@users.noreply.github.com>
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.

4 participants