Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(66)

Issue 45741002: trytond: Sort keys when dumping data

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 3 days ago by ced
Modified:
1 week, 3 days ago
Reviewers:
reviewbot
Visibility:
Public.

Description

This ensure stability for future comparison. issue7761

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M trytond/ir/model.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 2
ced
1 week, 3 days ago (2018-10-05 20:57:22 UTC) #1
reviewbot
1 week, 3 days ago (2018-10-05 21:04:08 UTC) #2
https://codereview.tryton.org/45741002/diff/1/trytond/ir/model.py#newcode98
trytond/ir/model.py:98:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/45741002/diff/1/trytond/ir/model.py#newcode270
trytond/ir/model.py:270:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/45741002/diff/1/trytond/ir/model.py#newcode495
trytond/ir/model.py:495:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/45741002/diff/1/trytond/ir/model.py#newcode674
trytond/ir/model.py:674:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/45741002/diff/1/trytond/ir/model.py#newcode674
trytond/ir/model.py:674:
E203 whitespace before ','

URL: https://codereview.tryton.org/45741002
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 0147766