From c68b76f8434ac20f937696421e9a5315475d0085 Mon Sep 17 00:00:00 2001 From: Alexander Schepanovski Date: Tue, 27 Aug 2019 15:02:21 +0700 Subject: [PATCH 1/2] ui: remove "Reproducing ..." message It as always followed by a more specific one, so it's superfluous. --- dvc/stage.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/dvc/stage.py b/dvc/stage.py index f91aff25d..142e94094 100644 --- a/dvc/stage.py +++ b/dvc/stage.py @@ -327,8 +327,6 @@ class Stage(object): if interactive and not prompt.confirm(msg): raise DvcException("reproduction aborted by the user") - logger.info("Reproducing '{stage}'".format(stage=self.relpath)) - self.run(**kwargs) logger.debug("'{stage}' was reproduced".format(stage=self.relpath)) From b52265a1d28c6b89ae23e6bdfd1ea0b0c0f25bb8 Mon Sep 17 00:00:00 2001 From: Alexander Schepanovski Date: Tue, 27 Aug 2019 15:04:37 +0700 Subject: [PATCH 2/2] ui: remove extraneous stage.save() stage.update() already does that via .run(). That was causing printing a confusing message to user. --- dvc/repo/update.py | 1 - 1 file changed, 1 deletion(-) diff --git a/dvc/repo/update.py b/dvc/repo/update.py index 1eb352c21..bf97fca4e 100644 --- a/dvc/repo/update.py +++ b/dvc/repo/update.py @@ -4,6 +4,5 @@ def update(self, target): stage = Stage.load(self, target) with self.state: stage.update() - stage.save() stage.dump()