erichexter / twitter.bootstrap.mvc

nuget package to make bootstrap easy with mvc4
Apache License 2.0
248 stars 134 forks source link

Examples should not mix with twitter.bootstrap.mvc4 package #66

Open serra opened 11 years ago

serra commented 11 years ago

I run most of my tests using the sample package, so I only just noticed that there is a packaging error - we can no longer simply add all files like here in App_Start. There are probably other places that have similar errors, but right now I don't have time to check it out (controllers? example routes?) so instead created this issue.

Before packaging and pushing to NuGet, It's probably best to do a test that installs the packages without the sample and check that everything compiles out-of-the-box.

erichexter commented 11 years ago

agreed

Eric Hexter

blog | http://Hex.LosTechies.com info | http://www.linkedin.com/in/erichexter

On Fri, Feb 1, 2013 at 9:00 AM, Marijn van der Zee <notifications@github.com

wrote:

I run most of my tests using the sample package, so I only just noticed that there is a packaging error - we can no longer simply add all files like here in App_Starthttps://github.com/erichexter/twitter.bootstrap.mvc/blob/master/src/twitter-bootstrap-mvc.nuspec#L28. There are probably other places that have similar errors, but right now I don't have time to check it out (controllers? example routes?) so instead created this issue.

Before packaging and pushing to NuGet, It's probably best to do a test that installs the packages without the sample and check that everything compiles out-of-the-box.

— Reply to this email directly or view it on GitHubhttps://github.com/erichexter/twitter.bootstrap.mvc/issues/66.