-
Notifications
You must be signed in to change notification settings - Fork 10
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
Coverage #28
Conversation
@edewit 👀 |
@edewit since Appveyor is using MS VS 12.0 maybe moving MSTestFramework to 12.0 will help getting rid of " Could not find SDK "MSTestFramework, Version=14.0" I've also noticed that we have more warning on undocumented filed since you move them from private to public. maybe a separate PR to remove all warnings. |
@edewit no luck, i'm closing PR |
@corinnekrych your suggestion worked now it builds successful |
@edewit Just did a PR on your branch to remove some warnings. |
…entation.missing.warning Add XML doc to clear warning from appveyor CI build
what is wrong with the build? |
@edewit +1 to merge |
Changed to use test that use mocks instead of running node server, target windows 8.1 and 8.0 xamarin ios and android