Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

New flags readme #422

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

Closed
nol166 wants to merge 7 commits into master from new-flags-readme
Closed

New flags readme #422

nol166 wants to merge 7 commits into master from new-flags-readme

Conversation

Copy link
Contributor

@nol166 nol166 commented Apr 3, 2019

Describe in detail the problem you had and how this PR fixes it

Update quick start guide and readme based on new flags introduced with #420

Adds a clause in bug report template about ide.coder.com issues

Is there an open issue you can link to?

n/a

John McCambridge added 2 commits April 3, 2019 17:35
... configuration options, update bug template to direct ide.coder.com issues to the correct place.
@nol166 nol166 requested a review from kylecarbs April 3, 2019 22:37
@nol166 nol166 requested a review from code-asher as a code owner April 3, 2019 22:37
-h, --help output usage information
```
### Extension Directory
Point code-server at a local directory with installed extensions. This allows inheriting local VSCode extensions by specifying `--extensions-dir ~/.vscode/extensions`
Copy link
Member

@kylecarbs kylecarbs Apr 4, 2019

Choose a reason for hiding this comment

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

Feels a bit difficult to parse this. Maybe:

Specify a custom directory for extensions. Provides compatibility with local VS Code extensions by specifying `--extensions-dir ~/.vscode/extensions`.


### Data Directory
### User Data Directory
Specify the root folder that VS Code will start in. This allows inheriting local VS Code configuration. Example `--user-data-dir ~/.config/User`
Copy link
Member

@kylecarbs kylecarbs Apr 4, 2019

Choose a reason for hiding this comment

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

Maybe:

Specify where data is stored. Provides compatibility with local VS Code configuration by specifying `--user-data-dir ~/.config/User`.

Copy link

Choose a reason for hiding this comment

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

I realize this is correct for Linux, but for mac (at least on my installation), the local VS Code configuration is stored in --user-data-dir ~/Library/Application\ Support/Code/
It'd be nice to list both.

nol166 reacted with thumbs up emoji
README.md Outdated
How to [secure your setup](/doc/security/ssl.md).
### Customize User Data
- Use your local VSCode configuration and locally installed extensions with the `--user-data-dir` and `--extensions-dir` flags. View the [quick start](doc/self-hosted/index.md) guide for more.
Copy link
Member

@kylecarbs kylecarbs Apr 4, 2019

Choose a reason for hiding this comment

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

Maybe:

Use VS Code Data

Use your existing VS Code configuration and extensions using the --user-data-dir and --extensions-dir flags. View the quick start guide for more.

@nol166 nol166 requested a review from kylecarbs April 4, 2019 17:24
Copy link
Contributor

The readme has the "-h" flag twice. First for (-h, --host) and second for (-h, --help) causing a conflict as -h will never trigger for the help menu. This conflict is also found in the latest release.

Copy link
Contributor

Just noticed doc/self-hosted/index.md also mentions the code-server -h help option at bottom.

Use code-server -h or code-server --help to view the usage for the cli. This is also shown at the beginning of this section.

should read:
Use code-server --help to view the usage for the cli. This is also shown at the beginning of this section.

nol166 reacted with thumbs up emoji

Copy link
Member

I've updated this in the telemetry PR

@kylecarbs kylecarbs deleted the new-flags-readme branch April 17, 2019 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@code-asher code-asher Awaiting requested review from code-asher

@kylecarbs kylecarbs Awaiting requested review from kylecarbs

1 more reviewer

@joelhock joelhock joelhock left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /