Fix git cl format following removal of Rietveld.
Commit a185e2e3c83d805d3306a3f1fe9258ae2e75f231 removed the Rietveld code review
implementation, albeit hasn't expunged all references to Rietveld. This leads
to an assertion error when running |git cl format| on a non-Gerrit change.
There appears to have been a recent attempt to articulate this error better in
a185e2e3c83d805d3306a3f1fe9258ae2e75f231, albeit not fix the underlying issue.
This change addresses the immediate issue by defaulting the code review
implementation to Gerrit rather than Reitveld.
R=joenotcharles@google.com, tandrii@chromium.org
Bug: 914150
Change-Id: I64d33e5a172cc43339ec417f7f0a7820e0337772
Reviewed-on: https://chromium-review.googlesource.com/c/1372928
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Bryce Thomas <bryct@amazon.com>
diff --git a/git_cl.py b/git_cl.py
index eb5ba35..549aae2 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -1133,9 +1133,9 @@
self.issue = int(issue)
return
- # No issue is set for this branch, so decide based on repo-wide settings.
+ # No issue is set for this branch, so default to gerrit.
return self._load_codereview_impl(
- codereview='gerrit' if settings.GetIsGerrit() else 'rietveld',
+ codereview='gerrit',
**kwargs)
def IsGerrit(self):