Skip to content

Feature/update links commands #255

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

Merged
merged 3 commits into from
Mar 27, 2019

Conversation

carlosazaustre
Copy link
Contributor

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

Updated download links and added tar command to extract the files.
Also, I added the command for

Is there an open issue you can link to?

None

@NGTmeaty
Copy link
Contributor

For the download link, I believe we'll want to show users to the latest version.

@illiapoplawski
Copy link
Contributor

illiapoplawski commented Mar 18, 2019

A copy/paste command for downloading the latest linux release is:
curl -s https://api.github.com/repos/codercom/code-server/releases/latest | grep "browser_download_url.*code-server-.*linux-x64.tar.gz" | cut -d '"' -f 4 | wget -O ~/code-server-linux.tar.gz -qci -

mkdir -p ~/code-server-linux && tar -xzvf code-server-linux.tar.gz --strip 1 -C $_

Would make it easier for people to download without needing to manually edit the commands

@NGTmeaty
Copy link
Contributor

I do like what @illiapoplawski posted. From short amount of research I've done, the one posted above does seem to be the most streamlined approach to getting the latest release.

edit: looking into this a bit more, looks like we agreed on /releases/latest at #130

@illiapoplawski
Copy link
Contributor

@NGTmeaty #130 and my command both use /releases/latest however #130 requires the user to manually enter the version to download. My command automatically grabs the latest version requiring no editing of the command by the user

@NGTmeaty
Copy link
Contributor

Yup I saw. I was just pointing out the consensus we reached last time.

@kylecarbs
Copy link
Member

There's a merge conflict to fix. Once fixed, I'd be happy to merge.

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