Add auto install script by nhooyr · Pull Request #1701 · coder/code-server · GitHub
Skip to content

Add auto install script - #1701

Merged
nhooyr merged 27 commits into
masterfrom
auto-install
May 28, 2020
Merged

Add auto install script#1701
nhooyr merged 27 commits into
masterfrom
auto-install

Conversation

@nhooyr

@nhooyr nhooyr commented May 20, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@nhooyr
nhooyr force-pushed the auto-install branch 3 times, most recently from c2230c4 to 7e50de2 Compare May 20, 2020 13:11
@nhooyr

nhooyr commented May 20, 2020

Copy link
Copy Markdown
Contributor Author

@nhooyr
nhooyr requested review from code-asher and kylecarbs May 20, 2020 13:16
@nhooyr
nhooyr force-pushed the auto-install branch 2 times, most recently from b6684bd to a3033bc Compare May 20, 2020 13:18
@nhooyr

nhooyr commented May 20, 2020

Copy link
Copy Markdown
Contributor Author

Need to add to README.md and guide.md

In the next release we'll add installation over SSH support and then deprecate sshcode.

@nhooyr
nhooyr force-pushed the auto-install branch 2 times, most recently from 18164f7 to 4e8b1b8 Compare May 20, 2020 13:21
@nhooyr
nhooyr marked this pull request as ready for review May 20, 2020 13:29
@nhooyr
nhooyr requested a review from coadler May 20, 2020 15:47
@nhooyr

nhooyr commented May 20, 2020

Copy link
Copy Markdown
Contributor Author
  • I think it'd be good to switch to long style flags and remove getopts.

@nhooyr

nhooyr commented May 20, 2020

Copy link
Copy Markdown
Contributor Author
  • Also good idea to remove the hard coded version and always fetch latest in case people download and store the script.

@code-asher code-asher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is dope. Love the dry run.

Comment thread install.sh
Comment thread install.sh Outdated
Comment thread install.sh Outdated
Comment thread install.sh Outdated
@code-asher

Copy link
Copy Markdown
Member

Agreed about the long option names, those are my preference in general. Fetching the latest version also seems like a good idea.

@nhooyr

nhooyr commented May 21, 2020

Copy link
Copy Markdown
Contributor Author

And check for libc and libcxx #1706

@nhooyr
nhooyr requested a review from code-asher May 21, 2020 19:34
@nhooyr

nhooyr commented May 21, 2020

Copy link
Copy Markdown
Contributor Author

@code-asher Please re-review the new changes.

Last thing I need to take care of is #1706

@nhooyr

nhooyr commented May 21, 2020

Copy link
Copy Markdown
Contributor Author

More clear as discussed in PR.
Comment thread install.sh Outdated
@nhooyr
nhooyr merged commit 80bcfd9 into master May 28, 2020
@nhooyr
nhooyr deleted the auto-install branch May 28, 2020 00:35
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