It's mostly OK, but some of the vertical space changes are excessive in 
particular gncinvoice_jinja.py line 213 and latex_invoices.py line 65.

I don't much care for the way it handled gncinvoice_jinja.py line 207 either: 
I'd prefer to move the comment above the line and if it's still too wide drop 
the whole call, like so:
     # something like 2004-11-01
     filename_date = 
         invoice.GetDatePosted().strftime("%Y-%m-%d")

But that's probably too much to ask of an automated tool.

Regards,
John Ralls

> On Jul 6, 2020, at 1:10 PM, c.holterm...@gmx.de wrote:
> 
> Hello folks,
> 
> I'd like to use a code formatter on the python bindings. I tried black
> [1] and liked what I saw. Would you be ok with reformatting the whole
> python bindings with it?
> I used it on two example scripts:
> https://github.com/Gnucash/gnucash/pull/736/commits/0434acbe1035ed679d23242a412c48a680ac5a07
> 
> 
> regards,
> 
> Christoph Holtermann
> 
> Links:
> ------
> [1] https://github.com/psf/black
> _______________________________________________
> gnucash-devel mailing list
> gnucash-devel@gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-devel

_______________________________________________
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

Reply via email to