Re: Considering removing support for ("iLmsu") regex groups in URLpatterns. Do you use them?

2016-12-18 Thread Tim Graham
Only case insensitive matching was tested in the URL tests and none of this is documented. That's the only flag where I see a straightforward use case (but I avoid regexes of any complexity and didn't even know what the flags were for until I just looked them up), even if case-insensitive URLs a

Re: Considering removing support for ("iLmsu") regex groups in URLpatterns. Do you use them?

2016-12-18 Thread Adam Johnson
Since they were used in several places in Django's test suite I feel like it's highly likely they're out there in use in the wild. Also if Django were to remove it, it would both 1) be a bit surprising compared to the re module, as it's an implementation detail that the urlparser prefixes '^/' an

Re: Please help me with "Writing your first patch for Django"

2016-12-18 Thread Adam Johnson
> > That appears to be the full stack trace since it errors out in the assert >> :D > > Oh yes, my bad, I'm used to pytest ;) > On Sunday, December 18, 2016 at 2:15:54 PM UTC, Florian Apolloner wrote: > > > > On Saturday, December 17, 2016 at 7:08:23 PM UTC+1, Adam Johnson wrote: >> >> Also you

Re: Please help me with "Writing your first patch for Django"

2016-12-18 Thread Florian Apolloner
On Saturday, December 17, 2016 at 7:08:23 PM UTC+1, Adam Johnson wrote: > > Also you might be experiencing failures specific to your operating system > and database setup - if you'd paste the stack traces we might be able to > figure more out. > That appears to be the full stack trace since it