From a927a506a75d6d2991324ad2b6f371ce9844f7da Mon Sep 17 00:00:00 2001 From: Jorge Orpinel Date: Thu, 27 Jun 2019 00:22:10 -0500 Subject: [PATCH] command: remove `default` property from "targets" argparse code in data sync and `commit` commands, since `None` is the default value for the removed `default` property, see https://docs.python.org/dev/library/argparse.html#default --- dvc/command/commit.py | 1 - dvc/command/data_sync.py | 1 - 2 files changed, 2 deletions(-) diff --git a/dvc/command/commit.py b/dvc/command/commit.py index ef703e117..731b44bac 100644 --- a/dvc/command/commit.py +++ b/dvc/command/commit.py @@ -67,7 +67,6 @@ def add_parser(subparsers, parent_parser): commit_parser.add_argument( "targets", nargs="*", - default=None, help="DVC-files to commit. Optional." "(Finds all DVC-files in the workspace by default.)", ) diff --git a/dvc/command/data_sync.py b/dvc/command/data_sync.py index b8319b8d4..17bebcb38 100644 --- a/dvc/command/data_sync.py +++ b/dvc/command/data_sync.py @@ -116,7 +116,6 @@ def shared_parent_parser(): shared_parent_parser.add_argument( "targets", nargs="*", - default=None, help="Limit command scope to these DVC-files. " "Using -R, directories to search DVC-files in can also be given.", )