Merge pull request #3 from argoproj-labs/fix-logging

Only log when not dry run
argoproj-labs · Jan 9, 2020 · 5d95f05 · 5d95f05
2 parents fe3d8a7 + fec7d81
commit 5d95f05
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/gordian/gordian.py b/gordian/gordian.py
@@ -132,9 +132,9 @@ def apply_transformations(args, transformations):
             if not args.dry_run:
                 try:
                     repo.repo.create_pull(args.pr_message, '', 'master', repo.branch_name)
+                    logger.info(f'PR created: {args.pr_message}. Branch: {repo.branch_name}')
                 except GithubException as e:
-                    print(f'PR already exists for {repo.branch_name}')
-            logger.info(f'PR created: {args.pr_message}. Branch: {repo.branch_name}')
+                    logger.info(f'PR already exists for {repo.branch_name}')
 
 
 def main():

diff --git a/setup.py b/setup.py
@@ -3,7 +3,7 @@
 setup_reqs = ["pytest-cov", "pytest-runner", "flake8"]
 setuptools.setup(
     name="gordian",
-    version="0.2.0",
+    version="0.2.1",
     author="Intuit",
     author_email="cg-sre@intuit.com",
     description="A tool to search and replace YAML files in a Git repo",