On Mon, Dec 27, 2010 at 19:29, <[email protected]> wrote: > On Mon, Dec 27, 2010 at 10:23:19PM -0500, Augie Fackler wrote: > > On Dec 27, 2010, at 10:08 PM, [email protected] wrote: > > > On Mon, Dec 27, 2010 at 05:44:01PM -0800, Dave Borowitz wrote: > > >> Looks like only 3 of these got picked. Anything I need to change, or > have > > >> you just not gotten to the rest? > > > I tried to land the 4 that were obviously correct to me. I was meaning > to reply > > > about the other ones but hadn't gotten round to that yet. > > > > > >> On Tue, Dec 21, 2010 at 14:50, <[email protected]> wrote: > > >>> I've been meaning to help knock some bugs off the tracker for a > while, > > >>> so here are some easy fixes. > > >>> > > >>> There are a number of other outstanding bugs I'm still working on, > but > > >>> which will require significantly more code. > > >>> > > >>> ebc736f test_repository: Add setUp/tearDown to CreateRepositoryTests. > > > Is this preparing for more tests in that TestCase that hit disk, or > perhaps > > > just simplifying the test method? > > > > > >>> 0c56328 repo: Allow recursive mkdir in Repo.init(_bare). > > > I wonder if this is really the right layer for this kind of thing. > Wouldn't it make > > > more sense to put that sort of thing in the UI layer? > > > > > >>> 8cc2aee client: Standardize on single quotes. > > > I'd like to land this patch, but I also think we should decide on a > > > standardized way to deal with quotes. Should we always just use single > quotes and > > > only use """-style for docstrings? IIRC PEP8 doesn't say anything about > this, > > > so we should probably update HACKING with whatever we decide. > > The general rule I've observed (can't remember if it's PEP8 or not, but > I'm over 90% confident it is) is that triple-quotes are always """, single > quotes are ' unless using " would result in less escaping within the string. > Does that seem reasonable (especially given that it's a common approach > overall?) > That works for me. Dave? >
Sounds great. I also don't know where this suggestion originally came from, but it's what I use on my code at work too, so it's the least cognitive load for me. > Cheers, > > Jelmer >
_______________________________________________ Mailing list: https://launchpad.net/~dulwich-users Post to : [email protected] Unsubscribe : https://launchpad.net/~dulwich-users More help : https://help.launchpad.net/ListHelp
