-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
VS2015 migration #956
VS2015 migration #956
Conversation
e23d4bd
to
bcef9b6
Compare
Nice! I know @Eilon was just asking me if we could switch Octokit.net to VS 2015. I think he even offered to help. 😛 |
@haacked it looks like the xml-doc warnings are causing the build to fail incorrectly. I really don't want to go through and triage it all, but maybe I have to? |
I would set the doc warnings as warnings and not errors and assign it to me to fix later. |
@haacked good point, I'll do that now and see if the build is happy with that... |
3f96554
to
bfb0559
Compare
Looks to me! |
👍 |
netcore45
->netcore451
target it any more