-
Notifications
You must be signed in to change notification settings - Fork 63
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
Migrate to Automatic Package Restore #373
Comments
I've almost merged this. The only issue I'm seeing is that there is a discrepancy between building from VS and the command line. If you do a fresh clone of your repo, build from within VS and run the demo web site, then checking out the log page produces the following error:
If you take another fresh clone (or clean the existing one with Could you take a look at this? Thanks! Reported by @atifaziz on 2014-10-12 22:57:02
|
Reported by @atifaziz on 2014-10-12 22:57:26
|
I also noticed that P.S. Don't forget to turn reviewing on on the clone repo. Reported by @atifaziz on 2014-10-13 06:32:38 |
Downloading
Reported by @atifaziz on 2014-10-13 06:35:34 |
For a potential fix to the above comment, see the following Web Essentials 2013 issue that's in the same vein: madskristensen/WebEssentials2013#1293 Reported by @atifaziz on 2014-10-13 06:46:11 |
I've merged changes thus far (revision fbc19da53512). Reported by @atifaziz on 2015-03-04 08:09:31 |
Addressed 3rd comment: This concludes the review. Reported by @atifaziz on 2015-03-04 08:16:31
|
Reported by @atifaziz on 2015-03-04 10:49:53
|
https://code.google.com/r/jamesdriscoll71-nuget-changes
Purpose of code changes on this branch:
Change the way we do NuGet in line with:
http://blog.davidebbo.com/2014/01/the-right-way-to-restore-nuget-packages.html
When reviewing my code changes, please focus on:
After the review, I'll merge this branch into:
/trunk
Originally reported on Google Code with ID 373
Reported by @JamesDriscoll on 2014-10-12 09:59:35
The text was updated successfully, but these errors were encountered: