Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Issue 83603002: Improve and refactor push-to-trunk script. (Closed)

Created:
7 years, 1 month ago by Michael Achenbach
Modified:
7 years ago
Reviewers:
Jakob Kummerow
CC:
v8-dev
Visibility:
Public.

Description

Improve and refactor push-to-trunk script. Let change log formatter squash title and bug reference. Repair wrongly indented first line in change log. Add automatic rietveld reload of commit messages to enable late corrections. BUG= R=jkummerow@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=18057

Patch Set 1 : #

Patch Set 2 : Add system test with a non-mocked call to rietveld. #

Patch Set 3 : Presubmit correction. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+108 lines, -57 lines) Patch
M tools/push-to-trunk/common_includes.py View 3 chunks +8 lines, -13 lines 0 comments Download
M tools/push-to-trunk/push_to_trunk.py View 5 chunks +24 lines, -1 line 0 comments Download
M tools/push-to-trunk/test_scripts.py View 1 2 14 chunks +76 lines, -43 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Michael Achenbach
PTAL
7 years, 1 month ago (2013-11-22 14:56:25 UTC) #1
Jakob Kummerow
lgtm
7 years ago (2013-11-25 12:34:00 UTC) #2
Michael Achenbach
7 years ago (2013-11-25 14:20:47 UTC) #3
Message was sent while issue was closed.
Committed patchset #3 manually as r18057 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698