On 07/16/2015 01:10 PM, Mathieu Lirzin wrote: > Hi, > > I've not test it but I have some stylistic remarks :). Most of them apply > to other parts of your patch and to the other patches. >
Oh, Indeed, I screwed up the indentation everywhere :/ I'll fix it when resending the patches if it is needed. > Cyril Roelandt <tipec...@gmail.com> writes: > >> + (home-page "http://github.com/ActiveState/appdirs";) >> + (synopsis >> + "A small Python module for determining appropriate \" + >> \"platform-specific dirs, e.g. a \"user data dir\".") >> + (description >> + "A small Python module for determining appropriate \" + >> \"platform-specific dirs, e.g. a \"user data dir\".") > > I guess this is something not intentional. > To be honest, I have no idea of what this package does, so I thought I'd just use the synopsis as a description :) Cyril.
