Skip to content

use lighter-weight node image#518

Merged
metrue merged 2 commits intometrue:masterfrom
bard:master
Apr 28, 2020
Merged

use lighter-weight node image#518
metrue merged 2 commits intometrue:masterfrom
bard:master

Conversation

@bard
Copy link
Contributor

@bard bard commented Apr 27, 2020

Issue: #517
Summary: replace full (debian-based) node image with lighter (alpine-based) node image

The checklist before PR is ready for review:

  • has unit testing for new added codes
  • has functional testing for new added features
  • has checked the lint or style issues (*)
  • README updated if need

(*) make lint currently fails for me on unrelated files

@metrue
Copy link
Owner

metrue commented Apr 28, 2020

never mind, I am fixing the lint issue in this PR #519

Thanks a lot @bard

@codecov
Copy link

codecov bot commented Apr 28, 2020

Codecov Report

Merging #518 into master will decrease coverage by 0.69%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #518      +/-   ##
==========================================
- Coverage   25.46%   24.77%   -0.70%     
==========================================
  Files          54       54              
  Lines        1873     1873              
==========================================
- Hits          477      464      -13     
- Misses       1317     1335      +18     
+ Partials       79       74       -5     
Impacted Files Coverage Δ
container_runtimes/docker/sdk/docker.go 36.02% <0.00%> (-9.56%) ⬇️

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 16d20b2...06409fa. Read the comment docs.

@metrue metrue merged commit 8741db2 into metrue:master Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants