Skip to content

chore: update Docker image publish workflow #4847

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Feb 11, 2022
Merged

chore: update Docker image publish workflow #4847

merged 4 commits into from
Feb 11, 2022

Conversation

jawnsy
Copy link
Contributor

@jawnsy jawnsy commented Feb 10, 2022

  • Use official action to log in to Docker Hub
  • Run using pinned ubuntu-20.04 environment

@jawnsy
chore: update Docker image publish workflow
05fc4cf 
* Use official action to log in to Docker Hub
* Run using pinned ubuntu-20.04 environment
@jawnsy jawnsy self-assigned this Feb 10, 2022
@jawnsy jawnsy temporarily deployed to CI February 10, 2022 21:18 Inactive
@jawnsy jawnsy temporarily deployed to CI February 10, 2022 21:22 Inactive
@github-actions
Copy link

github-actions bot commented Feb 10, 2022 •
edited
Loading

✨ Coder.com for PR #4847 deployed! It will be updated on every commit.

@codecov
Copy link

codecov bot commented Feb 10, 2022 •
edited
Loading

Codecov Report

Merging #4847 (3dd0901) into main (177f0ed) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4847   +/-   ##
=======================================
  Coverage   69.17%   69.17%           
=======================================
  Files          29       29           
  Lines        1645     1645           
  Branches      363      363           
=======================================
  Hits         1138     1138           
  Misses        430      430           
  Partials       77       77

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 177f0ed...3dd0901. Read the comment docs.

@jawnsy jawnsy temporarily deployed to CI February 10, 2022 22:33 Inactive
@jawnsy jawnsy requested a review from jsjoeio February 10, 2022 22:38
@jawnsy jawnsy marked this pull request as ready for review February 10, 2022 22:38
@jawnsy jawnsy requested a review from a team February 10, 2022 22:38
jsjoeio
jsjoeio approved these changes Feb 10, 2022
View reviewed changes
Copy link
Contributor

@jsjoeio jsjoeio left a comment

Choose a reason for hiding this comment

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

Thank you @jawnsy!

jsjoeio
jsjoeio approved these changes Feb 10, 2022
View reviewed changes
Copy link
Contributor

@jsjoeio jsjoeio left a comment

Choose a reason for hiding this comment

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

Thank you @jawnsy!

@jawnsy jawnsy temporarily deployed to CI February 10, 2022 23:17 Inactive
@jawnsy jawnsy merged commit 8135d2e into main Feb 11, 2022
@jawnsy jawnsy deleted the jawnsy/chore-update-docker-publish branch February 11, 2022 01:40
TinLe pushed a commit to TinLe/code-server that referenced this pull request Apr 23, 2022
@jawnsy @TinLe
chore: update Docker image publish workflow (coder#4847)
0e83a30 
* Use official action to log in to Docker Hub
* Run using pinned ubuntu-20.04 environment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsjoeio jsjoeio jsjoeio approved these changes

Assignees

@jawnsy jawnsy

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jawnsy @jsjoeio