-
-
Notifications
You must be signed in to change notification settings - Fork 794
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
Upgrade Castle.Core dependency to 4.0.0 #323
Conversation
Absolutely! Since Castle.Core is no longer -pre, we could go ahead and merge this to master and ship |
Thanks! |
Out of curiosity: do you bump major version on Moq when updating major version for a dependency? |
Not really. I'd like to bump the major version when there are major new features in Moq itself, so it's more meaningful for users. |
heya @jeremymeng, I pushed a merge from master>netcore to your branch. I dealt with all conflicts I found so far with master, but I'm having a few remaining errors:
Also, the xproj projects don't open in VS2017, which wants to make a one-way upgrade. If you feel that's fine, plz go ahead and accept the migration so we can have something people can open right after cloning in the latest & greatest VS :D. Finally, once these issues are resolved, we should be able to merge it to master and bump the version to make it stable. It should be just a matter of updating https://github.com/moq/moq4/blob/netcore/GitInfo.txt |
@kzu thanks for merging master to netcore and resolving the conflicts! I've fixed the build errors by including the the one new file added in master. I will spend some time to migrate the .xproj's. Just to be clear that after migration the projects will not work in VS 2015. I hope this it not a concern at all. |
Maybe we can do the migration in a separate branch? |
@kzu
Any plan to get a non-pre release out?
After VS 2017 RTM, we can try switching to the new .csproj for both .NET Full and .NET Core.