-
Notifications
You must be signed in to change notification settings - Fork 55
get unit tests running #236
Comments
@ctaggart what was the intent for |
@SeanFeldman The unit tests project that needs to get working is
|
There were some dependency complications before, but try it out and see if it is easier now. |
Tests now run, but they need to run each time on AppVeyor too. This is going to stay open until that happens. Failed tests should fail the build and block a merging a pull request. |
@ctaggart you had tests disabled in AppVeyor configuration. I've raised a new PR #252 to address it. |
@ctaggart this can be closed now? |
@chris5287, yes. Thank again @SeanFeldman. |
No description provided.
The text was updated successfully, but these errors were encountered: