search

zalando-stups / docker-node

DEPRECATED: Docker image with Node.js and Zalando CA
8 stars 9 forks source link

#46 add alpine builds #48

Closed luisfarzati closed 7 years ago

luisfarzati commented 7 years ago

Provides alpine alternative for both LTS and Current.

kgalli commented 7 years ago

@luisfarzati is there a particular reason why you added yarn here or is it just because it is the snippet used by the official node image. I kind of do not like to have npm used in some images and yarn in others.

We could also just follow the official node image to minimise the effort on our end. But I think we should agree on an approach.

luisfarzati commented 7 years ago

Agree on having npm for consistency. I just copied the original Dockerfile. I'll remove yarn.

kgalli commented 7 years ago

👍

luisfarzati commented 7 years ago

Removed yarn.-

kgalli commented 7 years ago

👍

luisfarzati commented 7 years ago

👍