Skip to content

Params: remove stage deps and --all option#1152

Merged
dmpetrov merged 4 commits into
masterfrom
params_all
Apr 17, 2020
Merged

Params: remove stage deps and --all option#1152
dmpetrov merged 4 commits into
masterfrom
params_all

Conversation

@dmpetrov
Copy link
Copy Markdown
Contributor

@dmpetrov dmpetrov commented Apr 15, 2020

Params:

  1. No dependencies to stage files anymore
  2. --all option for diff

Per treeverse/dvc/pull/3631

@dmpetrov dmpetrov requested a review from jorgeorpinel April 15, 2020 10:13
@shcheklein shcheklein temporarily deployed to dvc-landing-params-all-fdcdw3t April 15, 2020 10:13 Inactive
Copy link
Copy Markdown
Contributor

@jorgeorpinel jorgeorpinel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few minor details 👇 Thanks!

Comment thread content/docs/command-reference/params/diff.md Outdated
Comment thread content/docs/command-reference/params/diff.md Outdated
Comment thread content/docs/command-reference/params/index.md Outdated
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-landing-params-all-fdcdw3t April 15, 2020 20:41 Inactive
@shcheklein shcheklein temporarily deployed to dvc-landing-params-all-fdcdw3t April 16, 2020 18:17 Inactive
@shcheklein
Copy link
Copy Markdown
Contributor

@dmpetrov @efiop it looks we haven't updated completion scripts for params

@efiop should we make a checkbox after all?

@dmpetrov
Copy link
Copy Markdown
Contributor Author

@dmpetrov @efiop it looks we haven't updated completion scripts for params

@shcheklein could you please clarify?

@shcheklein
Copy link
Copy Markdown
Contributor

@dmpetrov disregard it here, please, I'll create a ticket on the dvc repo. it's not related to docs.

@dmpetrov dmpetrov merged commit 8e3044b into master Apr 17, 2020
@dmpetrov dmpetrov deleted the params_all branch April 17, 2020 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants