Skip to content
This repository was archived by the owner on Jul 20, 2018. It is now read-only.

Add npm loglevel environment variable #38

Merged
merged 2 commits into from
Mar 13, 2015

Conversation

Starefossen
Copy link
Member

This PR reverts #36 and adds NPM_CONFIG_LOGLEVEL environment variable according to discussion in docker-library/official-images#543.

This reverts commit a72f08a, reversing
changes made to 79e00fb.
@Starefossen Starefossen changed the title Set npm loglevel environment variable info Add npm loglevel environment variable Mar 10, 2015
@@ -4,6 +4,7 @@ FROM buildpack-deps:jessie
RUN gpg --keyserver pool.sks-keyservers.net --recv-keys 9554F04D7259F04124DE6B476D5A82AC7E37093B DD8F2338BAE7501E3DD5AC78C273792F7D83545D

ENV IOJS_VERSION 1.5.1
ENV NPM_CONFIG_LOGLEVEL info
Copy link
Contributor

Choose a reason for hiding this comment

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

If you move this up a few lines, the layer can be cached between version bumps. 😄

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice catch!

@pesho
Copy link
Contributor

pesho commented Mar 10, 2015

LGTM 👍

@retrohacker
Copy link
Contributor

LGTM as well

pesho added a commit that referenced this pull request Mar 13, 2015
Add npm loglevel environment variable
@pesho pesho merged commit 8c35508 into nodejs:master Mar 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants