search

ajbrown / angular-loggly-logger

An AngularJS module which sends your $log message to loggly, and provides a service for manually sending messages to loggly.
MIT License
32 stars 49 forks source link

bower install --allow-root #44

Closed ghost closed 8 years ago

ghost commented 8 years ago

Our dockerized CI environment fails to build.

During the npm install phase after angular-loggly-logger is downloaded bower install is run by the root user causing the error.

I don't think it hurts to add the --allow-root flag here.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 78.472% when pulling 8290d40c9cb2d3127a4f30f766a0125a0b75637a on spira:hotfix/npm-postinstall into 46fe57cf1d5d24bb52d328a2baea3a807a9f5867 on ajbrown:master.

ajbrown commented 8 years ago

@jeremysik let me get a second opinion on allow-root before pulling this in.