-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Comments
@kylecarbs @coderasher |
Ah that version tag may be incorrect. Not certain we have the default value set for it. |
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. |
@coderasher ci needs to build/push the Dockerfile. Shouldn't the version numbers come from the tag/branch? |
Is the Docker build supposed to be the latest code rather than the latest release? |
Latest release, so it'd only build/push on a tagged commit. |
Ahhh, I see what you're saying. |
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. |
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-100Steps to Reproduce
code-server --version
Output looks like
server/0.0.0 darwin-x64 node-v8.9.3
It probably should show the same version as the downloaded filename.
The text was updated successfully, but these errors were encountered: