Skip to content

Include Travis#29

Open
roboflank wants to merge 1 commit intokn9ts:developfrom
roboflank:develop
Open

Include Travis#29
roboflank wants to merge 1 commit intokn9ts:developfrom
roboflank:develop

Conversation

@roboflank
Copy link
Copy Markdown

@roboflank roboflank commented May 24, 2016

No description provided.

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

kn9ts commented May 24, 2016

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

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

@kn9ts kn9ts May 28, 2016

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

kn9ts commented Feb 10, 2017

@denzelwamburu Still awaiting your updates a few months later 😸

@roboflank
Copy link
Copy Markdown
Author

roboflank commented Feb 13, 2017

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