Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Add travis. #496

Merged
merged 6 commits into from
Nov 30, 2018
Merged

Add travis. #496

merged 6 commits into from
Nov 30, 2018

Conversation

robdodson
Copy link
Contributor

Check that PRs don't break the build command.

.travis.yml Outdated
@@ -0,0 +1,12 @@
language: node_js
node_js:
- "10.14.1"
Copy link
Contributor

Choose a reason for hiding this comment

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

">= 10"?

Copy link
Contributor

Choose a reason for hiding this comment

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

To match package.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sam had a thing that only worked on node v10.11 for a bit... let's see if that was fixed. I'll try v10

Copy link
Contributor Author

Choose a reason for hiding this comment

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

using "10" seems to work. I don't think travis supports >= so i'll just go with 10 for now.

@robdodson robdodson merged commit 5a9369a into master Nov 30, 2018
@robdodson robdodson deleted the add-travis branch November 30, 2018 19:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants