On 2017-12-20 15:55, Cédric Krier wrote:
> Hi,
> 
> I think we should remove the 80 cols limit on XML file.
> The rational is that XML file are indented by 4 spaces with at least 2
> indentation (<tryton/> and <data/>) but also XML is quite verbose.
> So we reach the limit very quickly and so we add line break which makes
> the XML more difficult to read. Also most of the time 'pyson' attribute
> already exceed the limit and it can not be break in multiple line.
> 
> I think we should be more flexible. For example the <record/> line with
> 'model' and 'id' is more readable when not split. But <menu/> is more
> readable if each attribute is on his own line.
> 
> So I think the style should be one line with all attributes or one
> attribute per line, but not allow mixed.

Here is the style update: https://codereview.tryton.org/43871002

-- 
Cédric Krier - B2CK SPRL
Email/Jabber: cedric.kr...@b2ck.com
Tel: +32 472 54 46 59
Website: http://www.b2ck.com/

-- 
You received this message because you are subscribed to the Google Groups 
"tryton-dev" group.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/tryton-dev/20171222143735.GK8635%40kei.

Reply via email to