Skip to content

Regular updates (late Feb)#1014

Merged
jorgeorpinel merged 15 commits into
masterfrom
2020-02-21
Mar 12, 2020
Merged

Regular updates (late Feb)#1014
jorgeorpinel merged 15 commits into
masterfrom
2020-02-21

Conversation

@jorgeorpinel
Copy link
Copy Markdown
Contributor

@jorgeorpinel jorgeorpinel commented Feb 22, 2020

@shcheklein shcheklein temporarily deployed to dvc-landing-2020-02-21-wgkziif February 22, 2020 00:50 Inactive
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-landing-2020-02-21-wgkziif February 24, 2020 06:51 Inactive
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-landing-2020-02-21-wgkziif February 28, 2020 00:13 Inactive
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-landing-2020-02-21-wgkziif February 28, 2020 06:51 Inactive
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-landing-2020-02-21-wgkziif March 4, 2020 05:59 Inactive
@jorgeorpinel jorgeorpinel changed the title Regular updates (Feb 21) Regular updates (Early March) Mar 9, 2020
@jorgeorpinel jorgeorpinel marked this pull request as ready for review March 9, 2020 22:36
@jorgeorpinel jorgeorpinel changed the title Regular updates (Early March) Regular updates (early March) Mar 9, 2020
@jorgeorpinel jorgeorpinel changed the title Regular updates (early March) Regular updates (late Feb) Mar 9, 2020
@jorgeorpinel jorgeorpinel requested a review from shcheklein March 10, 2020 18:38
@jorgeorpinel jorgeorpinel mentioned this pull request Mar 10, 2020
1 task
Comment thread public/static/docs/command-reference/cache/index.md Outdated
Copy link
Copy Markdown
Contributor

@shcheklein shcheklein left a comment

Choose a reason for hiding this comment

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

looks good, but please check my comment on the config, .gitignore - may be you can come with some better way to handle that?

show Output metric values.
add Tag file as a metric file.
modify Modify metric file options.
modify Modify metric file values.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Opened treeverse/dvc/pull/3471 to match this (and other output string updates).

Comment thread public/static/docs/command-reference/metrics/modify.md
@jorgeorpinel jorgeorpinel requested a review from shcheklein March 12, 2020 01:15
@jorgeorpinel jorgeorpinel merged commit fa9d376 into master Mar 12, 2020
@jorgeorpinel jorgeorpinel deleted the 2020-02-21 branch March 21, 2020 03:01
efiop pushed a commit to treeverse/dvc that referenced this pull request Mar 23, 2020
* metrics show: update -h output to match docs
per treeverse/dvc.org@2c34521

* metric: update argument desc per treeverse/dvc.org/pull/1014

* metrics: match treeverse/dvc.org/pull/1014 again...
using term "formatting" instead of "options"

* list: update help output to match treeverse/dvc.org/pull/1014
That's treeverse/dvc.org/pull/1014

* dvc: add docstring to open_by_relpath
It's a public fn after all. Similar to Repo class

* help: update cmd output strings to match with docs

* remote: update help output

* term: file descriptor -> file object

* metrics: update add and remove help outputs
per #3471 (review)

* remove: update help output
per #3471 (review)

* root: update help output
per #3471 (review)
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.

Does DVC support rsync remotes?

2 participants