Re: #7198 - Better error message when app is missing models.py

2011-09-13 Thread Russell Keith-Magee
On Tue, Sep 13, 2011 at 4:41 PM, Jannis Leidel wrote: > On 12.09.2011, at 22:44, Carl Meyer wrote: > >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> Hi Gary, >> >> On 09/12/2011 12:04 AM, Gary Wilson Jr. wrote: >>> I'm a fan of not requiring a models.py, as IMHO it

Re: #7198 - Better error message when app is missing models.py

2011-09-13 Thread Jannis Leidel
On 12.09.2011, at 22:44, Carl Meyer wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Hi Gary, > > On 09/12/2011 12:04 AM, Gary Wilson Jr. wrote: >> I'm a fan of not requiring a models.py, as IMHO it shouldn't be any >> different than other common files found in an app e.g. urls.py,

Re: #7198 - Better error message when app is missing models.py

2011-09-12 Thread Jacob Kaplan-Moss
On Mon, Sep 12, 2011 at 3:44 PM, Carl Meyer wrote: > Actually, I think there's generally consensus that requiring models.py > is not ideal. Yeah, that sounds about right. I get bitten by missing models.py all the time, and I should supposedly know better. It's lousy usability,

Re: #7198 - Better error message when app is missing models.py

2011-09-12 Thread Carl Meyer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi Gary, On 09/12/2011 12:04 AM, Gary Wilson Jr. wrote: > I'm a fan of not requiring a models.py, as IMHO it shouldn't be any > different than other common files found in an app e.g. urls.py, > templatetags dir, etc. If I don't need any models for

Re: #7198 - Better error message when app is missing models.py

2011-09-12 Thread Gary Wilson Jr.
On Fri, Sep 9, 2011 at 4:35 PM, Justin Lilly wrote: > Not sure why this particular ticket is marked as DDN, as it seems like > a no-brainer. The patch provides a more clear error message when a > user is attempting to load an app which doesn't have a models.py. > >

#7198 - Better error message when app is missing models.py

2011-09-09 Thread Justin Lilly
Not sure why this particular ticket is marked as DDN, as it seems like a no-brainer. The patch provides a more clear error message when a user is attempting to load an app which doesn't have a models.py. https://code.djangoproject.com/ticket/7198 https://github.com/django/django/pull/39 Happy to