- 13 Jul, 2017 1 commit
-
-
Andrii Shyshkalov authored
Also adds tests for the bug and for --json output. R=agable@chromium.org, phajdan@chromium.org Change-Id: I4e2208fdad8e23d48d27d0a354470336a7b86180 Reviewed-on: https://chromium-review.googlesource.com/570030Reviewed-by:
Paweł Hajdan Jr. <phajdan.jr@chromium.org> Commit-Queue: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
-
- 11 May, 2017 2 commits
-
-
Andrii Shyshkalov authored
Previously landed as: > Change-Id: Ieb6415d55e85b91f11f9052b0fd08cf982b64d51 > Reviewed-on: https://chromium-review.googlesource.com/501849 R=agable@chromium.org,machenbach@chromium.org Bug: 717504 Change-Id: Iede5c29ff4c317b68d8c2bca319edec140a176f5 Reviewed-on: https://chromium-review.googlesource.com/502908 Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org>
-
Andrii Shyshkalov authored
This reverts commit 28a5d5de. Reason for revert: breaks assumption in function for adding footers Original change's description: > Relax git_footers parsing to match that of Gerrit (JGit). > > R=agable@chromium.org > > Bug: 717504 > Change-Id: Ieb6415d55e85b91f11f9052b0fd08cf982b64d51 > Reviewed-on: https://chromium-review.googlesource.com/501849 > Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org> > Reviewed-by: Aaron Gable <agable@chromium.org> > TBR=agable@chromium.org,machenbach@chromium.org,tandrii@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Bug: 717504 Change-Id: I9b4d619b2972be8434aff9464f1959fbcb3abd32 Reviewed-on: https://chromium-review.googlesource.com/503030Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
-
- 10 May, 2017 1 commit
-
-
Andrii Shyshkalov authored
R=agable@chromium.org Bug: 717504 Change-Id: Ieb6415d55e85b91f11f9052b0fd08cf982b64d51 Reviewed-on: https://chromium-review.googlesource.com/501849 Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org>
-
- 01 May, 2017 1 commit
-
-
Aaron Gable authored
This CL wholly revamps the way presubmit_support adds CQ_INCLUDE_TRYBOTS lines in commit descriptions. In particular, when the CL is being uploaded to Gerrit, it uses our pre-existing support for manipulating git footers to make the whole process much simpler. R=tandrii@chromium.org Bug: 710547 Change-Id: I5858282a44c590f131021fa3820f1cb3f70ef620 Reviewed-on: https://chromium-review.googlesource.com/487831 Commit-Queue: Aaron Gable <agable@chromium.org> Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org>
-
- 27 Apr, 2017 1 commit
-
-
Aaron Gable authored
This allows inserted footers to be specified as either before or after other potentially-present footers. It has one slight behavior change (reflected in the tests): If after_keys is specified but *doesn't match* any pre-existing footers, then the behavior does *not* switch to "insert as early as possible". The behavior switch only happens if the after_keys actually match a footer. R=iannucci@chromium.org Bug: 710547 Change-Id: If557978fe9309785285056eb557acbdc87960bb2 Reviewed-on: https://chromium-review.googlesource.com/487606Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Aaron Gable <agable@chromium.org>
-
- 26 Apr, 2017 1 commit
-
-
Andrii Shyshkalov authored
R=agable@chromium.org,iannucci@chromium.org Before, this block of footers won't be recognized by git_footers: Bug: Change-Id: xxx because of empty value for "Bug:". This CL fixes this behavior. Bug: 715614 Change-Id: Iabe45bfc027fda15cbe0cd5ce9b883ce3b891220 Reviewed-on: https://chromium-review.googlesource.com/487963 Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org>
-
- 21 Dec, 2016 1 commit
-
-
agable authored
R=iannucci@chromium.org BUG=475320 Review-Url: https://codereview.chromium.org/2354313003
-
- 03 Jun, 2016 1 commit
-
-
tandrii@chromium.org authored
Motivation: git_cl should start understanding Gerrit's git footers, to avoid appending BUG= after Change-Id: Ixxx line. R=sergiyb@chromium.org BUG=614587 Review-Url: https://codereview.chromium.org/2028303006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@300691 0039d316-1c4b-4281-b951-d872f2087c98
-
- 23 May, 2016 1 commit
-
-
martiniss@chromium.org authored
Lets you get CL tags for a given CL. BUG=591172 Review-Url: https://codereview.chromium.org/1915833003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@300658 0039d316-1c4b-4281-b951-d872f2087c98
-
- 17 Mar, 2016 1 commit
-
-
tandrii@chromium.org authored
Special case when the change description has no footers, but looks like a footer. R=machenbach@chromium.org,andybons@chromium.org BUG=579176 Review URL: https://codereview.chromium.org/1812803002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@299343 0039d316-1c4b-4281-b951-d872f2087c98
-
- 04 Mar, 2016 2 commits
-
-
tandrii@chromium.org authored
BUG=579183 Review URL: https://codereview.chromium.org/1761743002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@299105 0039d316-1c4b-4281-b951-d872f2087c98
-
tandrii@chromium.org authored
This should insert the message according to Gerrit's own commit-msg hook implementation. BUG=579183 Review URL: https://codereview.chromium.org/1758943002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@299093 0039d316-1c4b-4281-b951-d872f2087c98
-
- 03 Sep, 2014 1 commit
-
-
luqui@chromium.org authored
BUG=407316 Review URL: https://codereview.chromium.org/521033002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@291777 0039d316-1c4b-4281-b951-d872f2087c98
-