Skip to content

code-server --version reports development in dockerfile #131

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
kennu opened this issue Mar 8, 2019 · 8 comments
Closed

code-server --version reports development in dockerfile #131

kennu opened this issue Mar 8, 2019 · 8 comments
Labels
bug Something isn't working

Comments

@kennu
Copy link

kennu commented Mar 8, 2019

First of all, this is an awesome project and I'm already using it to develop some stuff.

Version

  • code-server version: 1.31.1-100
  • OS Version: darwin-x64 (Mojave 10.14.2)

Steps to Reproduce

  1. code-server --version

  2. Output looks like

    server/0.0.0 darwin-x64 node-v8.9.3

It probably should show the same version as the downloaded filename.

@nhooyr
Copy link
Contributor

nhooyr commented Mar 8, 2019

@kylecarbs @coderasher

@nhooyr nhooyr added the bug Something isn't working label Mar 8, 2019
@kylecarbs
Copy link
Member

Ah that version tag may be incorrect. Not certain we have the default value set for it.

@code-asher
Copy link
Member

Partially fixed by 736feab. The versions come from Travis's build number, so I'm not sure how we'd get the versions in Docker unless we pull the latest binary instead of building it from scratch.

@nhooyr
Copy link
Contributor

nhooyr commented Mar 12, 2019

@coderasher ci needs to build/push the Dockerfile. Shouldn't the version numbers come from the tag/branch?

@code-asher
Copy link
Member

Is the Docker build supposed to be the latest code rather than the latest release?

@nhooyr
Copy link
Contributor

nhooyr commented Mar 12, 2019

Latest release, so it'd only build/push on a tagged commit.

@code-asher
Copy link
Member

Ahhh, I see what you're saying.

@kylecarbs kylecarbs changed the title code-server --version reports 0.0.0 code-server --version reports development in dockerfile Apr 24, 2019
@code-asher
Copy link
Member

code-asher commented Apr 25, 2019

Closing in favor of #58. We'll have to change the CI to push the Docker image in order to make it report the correct version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants