- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6.3k
chore: attempt to fix docker #5106
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
Conversation
| ✨ code-server docs for PR #5106 is ready! It will be updated on every commit. 
 | 
| Codecov Report
 @@           Coverage Diff           @@
##             main    #5106   +/-   ##
=======================================
  Coverage   71.33%   71.33%           
=======================================
  Files          30       30           
  Lines        1685     1685           
  Branches      374      374           
=======================================
  Hits         1202     1202           
  Misses        413      413           
  Partials       70       70           Continue to review full report at Codecov. 
 | 
Co-authored-by: Asher <ash@coder.com>
e3b5e72    to
    2380941      
    Compare
  
    | "package": "./ci/build/build-packages.sh", | ||
| "postinstall": "./ci/dev/postinstall.sh", | ||
| "publish:npm": "./ci/steps/publish-npm.sh", | ||
| "publish:docker": "./ci/steps/docker-buildx-push.sh", | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for consistency, i wanted it to be like the publish:npm flow/script.
* chore: attempt to fix docker * Update .github/workflows/docker.yaml Co-authored-by: Asher <ash@coder.com> * chore: add publish:docker to scripts Co-authored-by: Asher <ash@coder.com>
We had an issue publishing v4.0.3 where the action couldn't find the artifacts to download. This changes that flow to download the release assets from the release for the Docker workflow.