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.

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

[ready to merge] test in both linux and osx on travis #27

Merged
merged 6 commits into from
Sep 16, 2015
Merged

[ready to merge] test in both linux and osx on travis #27

merged 6 commits into from
Sep 16, 2015

Conversation

vielmetti
Copy link
Contributor

Adding test cases for Travis to build on osx. I expect some of these to fail, but it's worth a try.

an awkward way to get the [osx,linux]+[node versions] matrix, but
until the osx supports this by default, we have this. not sure it will work,
hence the test.
we're doing nvm in before_install instead.
@vielmetti
Copy link
Contributor Author

okay! we now have all systems green for Travis for all versions of node plus all supported operating systems, this should be ready to merge.

I could squash the commit, I guess, but the evolution is entertaining. I'm looking forward to support directly by Travis for Node on OS X.

@vielmetti vielmetti changed the title test in linux and osx on travis [ready to merge] test in both linux and osx on travis Sep 16, 2015
rla added a commit that referenced this pull request Sep 16, 2015
[ready to merge] test in both linux and osx on travis
@rla rla merged commit 967b02b into rla:master Sep 16, 2015
@rla
Copy link
Owner

rla commented Sep 16, 2015

Thank you! The PR has been merged and everything is green. I do not have my own access to OSX and it's good to see that it runs there as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants