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

Run tests in Windows OS #22

Merged
merged 1 commit into from
Feb 9, 2015
Merged

Run tests in Windows OS #22

merged 1 commit into from
Feb 9, 2015

Conversation

blond
Copy link
Contributor

@blond blond commented Feb 9, 2015

To run tests in Windows OS on every Pull Request or commit as well as using the Travis need use AppVeyor.

For this need to add configuration file appveyor.yml and add mock-fs to your projects in AppVeyor.

More read in docs.

@@ -0,0 +1,19 @@
environment:
matrix:
- nodejs_version: "0.8"
Copy link
Owner

Choose a reason for hiding this comment

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

At least on Travis, npm needs to be upgraded so that things work with dependencies using ^. Will that be an issue on AppVeyor as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, on AppVeyor npm also needs to be upgraded.

@tschaub
Copy link
Owner

tschaub commented Feb 9, 2015

I've set up AppVeyor to run on all branches. If you push a new commit, I assume a build will be triggered and both AppVeyor and Travis should report on the commit status.

Would you mind pushing a new version of 9d70eca so we could test? Or if things should work another way, let me know.

@blond
Copy link
Contributor Author

blond commented Feb 9, 2015

I tried to add commit to run tests in AppVeyor.

It started — https://ci.appveyor.com/project/tschaub/mock-fs/build/3.

@tschaub
Copy link
Owner

tschaub commented Feb 9, 2015

With Node 0.12 out, I'm happy to drop 0.8 from the Travis and AppVeyor runs if getting the right npm installed is a problem.

@blond
Copy link
Contributor Author

blond commented Feb 9, 2015

@tschaub
Copy link
Owner

tschaub commented Feb 9, 2015

Nice. Not fast, but free is good.

Thanks for the work on this @andrewblond.

tschaub added a commit that referenced this pull request Feb 9, 2015
@tschaub tschaub merged commit f1ad6fd into tschaub:master Feb 9, 2015
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.

2 participants