-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make travis more efficient #84
Conversation
tools/travis/build.sh
Outdated
@@ -3,7 +3,7 @@ | |||
|
|||
SCRIPTDIR=$(cd $(dirname "$0") && pwd) | |||
ROOTDIR="$SCRIPTDIR/../.." | |||
WHISKDIR="$ROOTDIR/openwhisk" | |||
WHISKDIR="$ROOTDIR/../openwhisk" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you need to change the location of where openwhisk is cloned to if you want to make this change. currently this is cloned by the setup.sh that is found in the openwhisk repo. you can create your own setup.sh if you want and clone it to a different location.
.travis.yml
Outdated
@@ -18,10 +18,7 @@ notifications: | |||
install: true | |||
|
|||
before_install: | |||
- 'if [ "$TRAVIS_PULL_REQUEST" = "false" ]; then openssl aes-256-cbc -K $encrypted_ff5bbb6225c5_key -iv $encrypted_ff5bbb6225c5_iv -in tests/credentials.json.enc -out tests/credentials.json -d; fi' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why are you removing the encryption. it needs this to get the credentials for the test case.
@@ -0,0 +1,9 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you are adding jshint validation you need to make sure the current code will pass. based on what I am seeing in my ide it will not
aef97ad
to
d619f66
Compare
@drcariel abandoned PR? Can I close it? |
No description provided.