Skip to content

refactor(ci): combine publish workflows #5471

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Aug 19, 2022
Merged

refactor(ci): combine publish workflows #5471

merged 2 commits into from
Aug 19, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Aug 17, 2022

No description provided.

@jsjoeio jsjoeio requested a review from a team August 17, 2022 22:30
@jsjoeio jsjoeio self-assigned this Aug 17, 2022
@jsjoeio jsjoeio added this to the August 2022 milestone Aug 17, 2022
@jsjoeio jsjoeio temporarily deployed to npm August 17, 2022 22:35 Inactive
@github-actions
Copy link

github-actions bot commented Aug 17, 2022

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

  • Last publish status: success
  • Commit: 486f70f

To install in a local project, run:

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

To install globally, run:

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

@codecov
Copy link

codecov bot commented Aug 17, 2022

Codecov Report

Merging #5471 (8db09dc) into main (9d8588b) will not change coverage.
The diff coverage is n/a.

❗ Current head 8db09dc differs from pull request most recent head 486f70f. Consider uploading reports for the commit 486f70f to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5471   +/-   ##
=======================================
  Coverage   72.44%   72.44%           
=======================================
  Files          30       30           
  Lines        1673     1673           
  Branches      366      366           
=======================================
  Hits         1212     1212           
  Misses        398      398           
  Partials       63       63           

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 9d8588b...486f70f. Read the comment docs.

Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful!

@code-asher
Copy link
Member

Oh we can delete the docker.yaml file now right?

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Aug 17, 2022

Oh we can delete the docker.yaml file now right?

Oops! Committing now.

@jsjoeio jsjoeio temporarily deployed to npm August 17, 2022 22:52 Inactive
@repo-ranger repo-ranger bot temporarily deployed to npm August 18, 2022 01:44 Inactive
@jsjoeio jsjoeio force-pushed the jsjoeio/workflows branch from 8db09dc to 486f70f Compare August 18, 2022 22:55
@jsjoeio jsjoeio temporarily deployed to npm August 18, 2022 23:01 Inactive
@jsjoeio jsjoeio merged commit 6a7c322 into main Aug 19, 2022
@jsjoeio jsjoeio deleted the jsjoeio/workflows branch August 19, 2022 15:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants