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

Unified Diff: recipe_modules/gclient/api.py

Issue 2270083004: Default gclient's GIT_MODE to True (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: A slightly gentler hand with pragmas Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | recipe_modules/gclient/config.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: recipe_modules/gclient/api.py
diff --git a/recipe_modules/gclient/api.py b/recipe_modules/gclient/api.py
index 595939ace6fb5cbfce780670026471dc0ccca148..7462e22c650c02fe69aec0978157f31b24ded516 100644
--- a/recipe_modules/gclient/api.py
+++ b/recipe_modules/gclient/api.py
@@ -153,7 +153,7 @@ class GclientApi(recipe_api.RecipeApi):
step_test_data = lambda: (
self.test_api.output_json(test_data_paths, cfg.GIT_MODE))
try:
- if not cfg.GIT_MODE:
+ if not cfg.GIT_MODE: # pragma: no cover
args = ['sync', '--nohooks', '--force', '--verbose']
if cfg.delete_unversioned_trees:
args.append('--delete_unversioned_trees')
@@ -239,7 +239,7 @@ class GclientApi(recipe_api.RecipeApi):
sync_step = None
try:
- if not cfg.GIT_MODE:
+ if not cfg.GIT_MODE: # pragma: no cover
try:
if revert:
self.revert(**kwargs)
@@ -266,7 +266,7 @@ class GclientApi(recipe_api.RecipeApi):
return sync_step
- def revert(self, **kwargs):
+ def revert(self, **kwargs): # pragma: no cover
"""Return a gclient_safe_revert step."""
# Not directly calling gclient, so don't use self().
alias = self.spec_alias
« no previous file with comments | « no previous file | recipe_modules/gclient/config.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698