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

Issue 50281002: trytond: Rename "install-dependences" into "activate-dependencies" (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 months ago by xcodinas
Modified:
3 months ago
Reviewers:
rietveld-bot, ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Remove not related change #

Patch Set 3 : Add changelog entry #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -7 lines) Patch
M CHANGELOG View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M trytond/admin.py View 1 chunk +1 line, -1 line 0 comments Download
M trytond/commandline.py View 1 chunk +2 lines, -2 lines 0 comments Download
M trytond/modules/__init__.py View 2 chunks +2 lines, -2 lines 0 comments Download
M trytond/pool.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 8
xcodinas
5 months ago (2018-04-20 09:07:25 UTC) #1
xcodinas
Remove not related change
5 months ago (2018-04-20 09:11:51 UTC) #2
reviewbot
https://codereview.tryton.org/50281002/diff/20001/trytond/admin.py#newcode45 trytond/admin.py:45: E712 comparison to True should be 'if cond is True:' or 'if cond:' ...
5 months ago (2018-04-20 09:34:31 UTC) #3
ced
Missing CHANGELOG entry
4 months, 2 weeks ago (2018-05-07 13:22:21 UTC) #4
xcodinas
Add changelog entry
4 months, 1 week ago (2018-05-09 07:48:33 UTC) #5
reviewbot
https://codereview.tryton.org/50281002/diff/40001/trytond/admin.py#newcode45 trytond/admin.py:45: E712 comparison to True should be 'if cond is True:' or 'if cond:' ...
4 months, 1 week ago (2018-05-09 08:13:09 UTC) #6
ced
LGTM
4 months, 1 week ago (2018-05-09 09:11:07 UTC) #7
rietveld-bot_tryton.org
4 months, 1 week ago (2018-05-09 13:32:29 UTC) #8
New changeset e99e39ab6a27 by Xavier Codinas in branch 'default':
Rename "install-dependences" into "activate-dependencies"
http://hg.tryton.org/trytond/rev/e99e39ab6a27
Sign in to reply to this message.

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