From 69982ccbcb814c7272896b128b33cb4ddd121499 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Tue, 10 Mar 2020 14:53:10 +0000 Subject: [PATCH 1/5] completion: bash: fix, update and sort --- scripts/completion/dvc.bash | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/scripts/completion/dvc.bash b/scripts/completion/dvc.bash index 135f706619..79284d307d 100644 --- a/scripts/completion/dvc.bash +++ b/scripts/completion/dvc.bash @@ -24,25 +24,25 @@ _dvc_destroy='-f --force' _dvc_diff='-t --show-json --show-hash' _dvc_fetch='-j --jobs -r --remote -a --all-branches -T --all-tags -d --with-deps -R --recursive' _dvc_fetch_COMPGEN=_dvc_compgen_DVCFiles -_dvc_get_url='' -_dvc_get='-o --out --rev --show-url' _dvc_gc='-a --all-branches --all-commits -T --all-tags -w --workspace -c --cloud -r --remote -f --force -p --projects -j --jobs' -_dvc_import_url='-f --file' +_dvc_get='-o --out --rev --show-url' +_dvc_get_url='' _dvc_import='-o --out --rev' +_dvc_import_url='-f --file' _dvc_init='--no-scm -f --force' _dvc_install='' _dvc_list='-R --recursive --outs-only --rev' _dvc_list_COMPGEN=_dvc_compgen_files _dvc_lock='$(compgen -f -X \!*?.dvc)' -_dvc_metrics='add modify rmeove show' +_dvc_metrics='add diff modify remove show' _dvc_metrics_add='-t --type -x --xpath' _dvc_metrics_add_COMPGEN=_dvc_compgen_files -_dvc_metrics_show='$(-t --type -x --xpath -a --all-branches -T --all-tags -R --recursive' -_dvc_metrics_show_COMPGEN=_dvc_compgen_files _dvc_metrics_diff='--targets -t --type -x --xpath -R --show-json' _dvc_metrics_modify='-t --type -x --xpath' _dvc_metrics_modify_COMPGEN=_dvc_compgen_files _dvc_metrics_remove='$(compgen -f)' +_dvc_metrics_show='$(-t --type -x --xpath -a --all-branches -T --all-tags -R --recursive' +_dvc_metrics_show_COMPGEN=_dvc_compgen_files _dvc_move='$(compgen -f)' _dvc_pipeline='list show' _dvc_pipeline_list='' @@ -64,8 +64,9 @@ _dvc_repro='-f --force -s --single-item -c --cwd -m --metrics --dry -i --interac _dvc_repro_COMPGEN=_dvc_compgen_DVCFiles _dvc_root='' _dvc_run='--no-exec -f --file -c --cwd -d --deps -o --outs -O --outs-no-cache --outs-persist --outs-persist-no-cache -m --metrics -M --metrics-no-cache -y --yes --overwrite-dvcfile --ignore-build-cache --remove-outs --no-commit -w --wdir' -_dvc_status='-j --jobs -r --remote -a --all-branches -T --all-tags -d --with-deps -c --cloud' _dvc_run_COMPGEN=_dvc_compgen_DVCFiles +_dvc_status='-j --jobs -r --remote -a --all-branches -T --all-tags -d --with-deps -c --cloud' +_dvc_status_COMPGEN=_dvc_compgen_DVCFiles _dvc_unlock_COMPGEN=_dvc_compgen_DVCFiles _dvc_unprotect_COMPGEN=_dvc_compgen_files _dvc_update='--rev' From e4959107a9ae861c19d5b70a848ebb856199e0d6 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Tue, 10 Mar 2020 14:57:34 +0000 Subject: [PATCH 2/5] completion: bash: fix missing `compgen`s --- scripts/completion/dvc.bash | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/completion/dvc.bash b/scripts/completion/dvc.bash index 79284d307d..4adf8d3c18 100644 --- a/scripts/completion/dvc.bash +++ b/scripts/completion/dvc.bash @@ -33,17 +33,17 @@ _dvc_init='--no-scm -f --force' _dvc_install='' _dvc_list='-R --recursive --outs-only --rev' _dvc_list_COMPGEN=_dvc_compgen_files -_dvc_lock='$(compgen -f -X \!*?.dvc)' +_dvc_lock=_dvc_compgen_DVCFiles _dvc_metrics='add diff modify remove show' _dvc_metrics_add='-t --type -x --xpath' _dvc_metrics_add_COMPGEN=_dvc_compgen_files _dvc_metrics_diff='--targets -t --type -x --xpath -R --show-json' _dvc_metrics_modify='-t --type -x --xpath' _dvc_metrics_modify_COMPGEN=_dvc_compgen_files -_dvc_metrics_remove='$(compgen -f)' +_dvc_metrics_remove=_dvc_compgen_files _dvc_metrics_show='$(-t --type -x --xpath -a --all-branches -T --all-tags -R --recursive' _dvc_metrics_show_COMPGEN=_dvc_compgen_files -_dvc_move='$(compgen -f)' +_dvc_move=_dvc_compgen_files _dvc_pipeline='list show' _dvc_pipeline_list='' _dvc_pipeline_show='-c --commands -o --outs --ascii --dot --tree -l --locked' From 4c95adeacce45eca8cbfb8a40882af11e23a08ff Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Tue, 10 Mar 2020 15:08:48 +0000 Subject: [PATCH 3/5] completion: bash: remove deprecated/hidden options --- scripts/completion/dvc.bash | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/completion/dvc.bash b/scripts/completion/dvc.bash index 4adf8d3c18..523dff7a6f 100644 --- a/scripts/completion/dvc.bash +++ b/scripts/completion/dvc.bash @@ -24,7 +24,7 @@ _dvc_destroy='-f --force' _dvc_diff='-t --show-json --show-hash' _dvc_fetch='-j --jobs -r --remote -a --all-branches -T --all-tags -d --with-deps -R --recursive' _dvc_fetch_COMPGEN=_dvc_compgen_DVCFiles -_dvc_gc='-a --all-branches --all-commits -T --all-tags -w --workspace -c --cloud -r --remote -f --force -p --projects -j --jobs' +_dvc_gc='-a --all-branches -T --all-tags -w --workspace -c --cloud -r --remote -f --force -p --projects -j --jobs' _dvc_get='-o --out --rev --show-url' _dvc_get_url='' _dvc_import='-o --out --rev' @@ -63,7 +63,7 @@ _dvc_remove_COMPGEN=_dvc_compgen_DVCFiles _dvc_repro='-f --force -s --single-item -c --cwd -m --metrics --dry -i --interactive -p --pipeline -P --all-pipelines --ignore-build-cache --no-commit -R --recursive --downstream' _dvc_repro_COMPGEN=_dvc_compgen_DVCFiles _dvc_root='' -_dvc_run='--no-exec -f --file -c --cwd -d --deps -o --outs -O --outs-no-cache --outs-persist --outs-persist-no-cache -m --metrics -M --metrics-no-cache -y --yes --overwrite-dvcfile --ignore-build-cache --remove-outs --no-commit -w --wdir' +_dvc_run='--no-exec -f --file -d --deps -o --outs -O --outs-no-cache --outs-persist --outs-persist-no-cache -m --metrics -M --metrics-no-cache --overwrite-dvcfile --ignore-build-cache --no-commit -w --wdir' _dvc_run_COMPGEN=_dvc_compgen_DVCFiles _dvc_status='-j --jobs -r --remote -a --all-branches -T --all-tags -d --with-deps -c --cloud' _dvc_status_COMPGEN=_dvc_compgen_DVCFiles From 3e8034411e6c77154c5a1627829935f1ea0245e3 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Tue, 10 Mar 2020 15:37:03 +0000 Subject: [PATCH 4/5] completion: bash: fix extra parenthesis - fixes metrics show -t --- scripts/completion/dvc.bash | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/completion/dvc.bash b/scripts/completion/dvc.bash index 523dff7a6f..4e46a8f6c7 100644 --- a/scripts/completion/dvc.bash +++ b/scripts/completion/dvc.bash @@ -41,7 +41,7 @@ _dvc_metrics_diff='--targets -t --type -x --xpath -R --show-json' _dvc_metrics_modify='-t --type -x --xpath' _dvc_metrics_modify_COMPGEN=_dvc_compgen_files _dvc_metrics_remove=_dvc_compgen_files -_dvc_metrics_show='$(-t --type -x --xpath -a --all-branches -T --all-tags -R --recursive' +_dvc_metrics_show='-t --type -x --xpath -a --all-branches -T --all-tags -R --recursive' _dvc_metrics_show_COMPGEN=_dvc_compgen_files _dvc_move=_dvc_compgen_files _dvc_pipeline='list show' From b14429910fa22e20c98f3060802000bb62f433a2 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Tue, 10 Mar 2020 15:51:28 +0000 Subject: [PATCH 5/5] completion: bash: re-add `gc --all-commits` Due to #3429 superceding #2681 --- scripts/completion/dvc.bash | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/completion/dvc.bash b/scripts/completion/dvc.bash index 4e46a8f6c7..61b1a93ebc 100644 --- a/scripts/completion/dvc.bash +++ b/scripts/completion/dvc.bash @@ -24,7 +24,7 @@ _dvc_destroy='-f --force' _dvc_diff='-t --show-json --show-hash' _dvc_fetch='-j --jobs -r --remote -a --all-branches -T --all-tags -d --with-deps -R --recursive' _dvc_fetch_COMPGEN=_dvc_compgen_DVCFiles -_dvc_gc='-a --all-branches -T --all-tags -w --workspace -c --cloud -r --remote -f --force -p --projects -j --jobs' +_dvc_gc='-a --all-branches --all-commits -T --all-tags -w --workspace -c --cloud -r --remote -f --force -p --projects -j --jobs' _dvc_get='-o --out --rev --show-url' _dvc_get_url='' _dvc_import='-o --out --rev'