-
Notifications
You must be signed in to change notification settings - Fork 90
Updating Nuget packages in some samples#3
Updating Nuget packages in some samples #3brgrz wants to merge 1 commit intoServiceStack:master from brgrz:master
Conversation
fixed two lines of code to work with latest SS
mythz
commented
May 5, 2013
thx, but there's a whitespace/new-line issue that shows this pull-request has modified every file.
You might have to re-clone from master to be able to send a clean pull-request.
brgrz
commented
May 5, 2013
hm, here the 56 modified files are depicted fine https://github.com/mhrovatic/ServiceStack.UseCases/commit/7d961bfdddbc5c71146846106e54ff0ca8125dd8
I don't know how to do it differently. I fork a project, clone my fork with GitHub for Windows, work on it, then commit and sync. Then I did pull request from the GitHub user interface.
mythz
commented
May 5, 2013
If it's not already, try again with autocrlf set to off, e.g:
git config --global core.autocrlf false
If the issue still exists, can you point to the lines that have changed and I can just apply the changes on my end.
brgrz
commented
May 5, 2013
Demis, I ran this locally but now I don't know how to make a new commit with this setting in place and a fresh pull request. You might want to update the code yourself.
The only thing I did is update all packages besides jQuery in the Manage Nuget packages dialog window and then add
using Servicestack;
to the usings list in Account and Home controllers so that ToRequestContext() was being found.
fixed two lines of code in CustomAuthenticationMvc to work with latest SS