Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Include Travis #29

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Include Travis #29

wants to merge 1 commit into from

Conversation

roboflank
Copy link

No description provided.

@roboflank roboflank mentioned this pull request May 24, 2016
Merged
@kn9ts
Copy link
Owner

kn9ts commented May 24, 2016

@denzelwamburu Thank you. I can use both, maybe 🎅 .

kn9ts
kn9ts reviewed May 28, 2016
View reviewed changes
.travis.yml
@@ -0,0 +1,7 @@
language: node_js
node_js:
- '0.10'
Copy link
Owner

Choose a reason for hiding this comment

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

Should only be tested from node version 4.3.2 and above

@kn9ts
Copy link
Owner

kn9ts commented May 28, 2016

@denzelwamburu Could also we add npm update -g npm so that it can be updated to the latest version

@kn9ts
Copy link
Owner

kn9ts commented May 28, 2016

Also am using these ENVIRONMENT configurations to facilitate the CI

HOST : localhost
PORT : 3000
EXPRESS_SESSION_KEY : 88186735405ab8d59f968ed4dab89da5515
ENDPOINT : https://safaricom.co.ke/mpesa_online/lnmo_checkout_server.php?wsdl
COVERALLS_REPO_TOKEN : EI2vRz1HRhJ3pGi7g3J6sMxI4dsnrWxtb
COVERALLS_SERVICE_NAME : travis-ci-org

@roboflank
Copy link
Author

roboflank commented May 29, 2016 •
edited
Loading

@kn9ts Probably This could be added to the test rather ? 😁 (npm update -g npm)

@kn9ts kn9ts force-pushed the develop branch 2 times, most recently from d1ec16b to 211d766 Compare September 12, 2016 18:22
@kn9ts
Copy link
Owner

kn9ts commented Feb 10, 2017

@denzelwamburu Still awaiting your updates a few months later 😸

@roboflank
Copy link
Author

Hey @kn9ts 😼 Did some edits a while back, seems i forgot to push them.
Let me update in a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@roboflank @kn9ts