Skip to content

Run tests in Windows OS - #22

Merged
tschaub merged 1 commit into
tschaub:masterfrom
blond:windows
Feb 9, 2015
Merged

Run tests in Windows OS#22
tschaub merged 1 commit into
tschaub:masterfrom
blond:windows

Conversation

@blond

@blond blond commented Feb 9, 2015

Copy link
Copy Markdown

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.

Comment thread appveyor.yml

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

tschaub commented Feb 9, 2015

Copy link
Copy Markdown
Owner

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

blond commented Feb 9, 2015

Copy link
Copy Markdown
Author

I tried to add commit to run tests in AppVeyor.

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

@tschaub

tschaub commented Feb 9, 2015

Copy link
Copy Markdown
Owner

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

blond commented Feb 9, 2015

Copy link
Copy Markdown
Author

@tschaub

tschaub commented Feb 9, 2015

Copy link
Copy Markdown
Owner

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