-
-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
@aarondandy I'll keep adding refactoring commits until you merge, so it makes sense if you review this commit by commit. Each is isolated and makes sense on it's own. |
… field and initializer
…rable and cleaned up inheritance design
…estore.Sources using a HashSet<string>
OK, that's my lot for today. I'm off to bed - have fun with my monster PR 😜 |
I have been following it incrementally today so it won't be too big. I'll give it another complete look and make sure the tests and dog*.bat still work for me. |
I don't think dog.bat will work anymore but we can also look at fixing that
|
Works great. I was able to get bau.bat and dog.debug.bat to work temporarily, they should be fine once the first package is pushed. For NCrunch I just had it copy NuGet.exe and NuGet.CommandLine.*.nupkg to the correct locations into the test folders. |
What did you have to do to get bau.bat working? That works fine for me as is. |
Maybe that one worked, I forget. From: Adam Ralphmailto:notifications@github.com What did you have to do to get bau.bat working? That works fine for me as is. Reply to this email directly or view it on GitHub: |
fixes #16
fixes #15