search

wantedly / step-pretty-slack-notify

Posts wercker build/deploy status to a Slack channel
MIT License
88 stars 34 forks source link

Fix a bug when using sudo #41

Closed spesnova closed 7 years ago

spesnova commented 7 years ago

What does this PR do?

Add -E option to sudo bundle exec $WERCKER_STEP_ROOT/run.rb command.

Motivation

To fix #39

Environment variables are not passed when using sudo.

REF

$ sudo -h | grep -e "-E"
  -E, --preserve-env          preserve user environment when running command
spesnova commented 7 years ago

@dtan4 @koudaiii Plz check this 😃

dtan4 commented 7 years ago

LGTM 👍 Thanks ❗️