Skip to content

Conversation

@jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Apr 14, 2022

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.

@jsjoeio jsjoeio requested a review from a team April 14, 2022 23:20
@github-actions
Copy link

github-actions bot commented Apr 14, 2022

✨ code-server docs for PR #5106 is ready! It will be updated on every commit.

@codecov
Copy link

codecov bot commented Apr 14, 2022

Codecov Report

Merging #5106 (2380941) into main (09fc64a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09fc64a...2380941. Read the comment docs.

@jsjoeio jsjoeio temporarily deployed to npm April 14, 2022 23:25 Inactive
@github-actions
Copy link

github-actions bot commented Apr 14, 2022

✨ code-server dev build published to npm for PR #5106!

  • Last publish status: success
  • Commit: 2380941

To install in a local project, run:

npm install @coder/code-server-pr@5106

To install globally, run:

npm install -g @coder/code-server-pr@5106

@jsjoeio jsjoeio temporarily deployed to npm April 14, 2022 23:33 Inactive
@code-asher code-asher self-requested a review April 14, 2022 23:38
@jsjoeio jsjoeio temporarily deployed to npm April 14, 2022 23:39 Inactive
@jsjoeio jsjoeio force-pushed the jsjoeio/temp-docker-fix branch from e3b5e72 to 2380941 Compare April 14, 2022 23:40
"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",
Copy link
Contributor Author

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.

@jsjoeio jsjoeio requested a review from code-asher April 14, 2022 23:41
@jsjoeio jsjoeio enabled auto-merge (squash) April 14, 2022 23:43
@jsjoeio jsjoeio self-assigned this Apr 14, 2022
@jsjoeio jsjoeio added ci Issues related to ci chore Related to maintenance or clean up labels Apr 14, 2022
@jsjoeio jsjoeio added this to the April 2022 milestone Apr 14, 2022
@jsjoeio jsjoeio temporarily deployed to npm April 14, 2022 23:45 Inactive
@jsjoeio jsjoeio merged commit dea6a40 into main Apr 15, 2022
@jsjoeio jsjoeio deleted the jsjoeio/temp-docker-fix branch April 15, 2022 00:02
TinLe pushed a commit to TinLe/code-server that referenced this pull request Apr 23, 2022
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Related to maintenance or clean up ci Issues related to ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants