Skip to content

cmd ref: describe update available output (for import stages) in status #477#1299

Merged
jorgeorpinel merged 6 commits into
treeverse:masterfrom
imhardikj:dvc-status
May 18, 2020
Merged

cmd ref: describe update available output (for import stages) in status #477#1299
jorgeorpinel merged 6 commits into
treeverse:masterfrom
imhardikj:dvc-status

Conversation

@imhardikj
Copy link
Copy Markdown
Contributor

This PR updates \static\docs\commands-reference\status.md with update available output for import stages to fix #477.

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.

Great start!

In addition to improving the description you added (see the one commend in this review, below), can you also please update one of the examples or add a new one that shows this "update available" case?

The best strategy for both things, so you're 100% sure you understand and thus explain what's happening, is to actually create a couple DVC repos (A and B) locally, dvc add some dummy data in A, use dvc import to bring it to B, update the data in A, and run dvc status in B. This way you will see exactly how DVC behaves under this situation and you'll be able to copy-paste the command output for the examples.

Please ping me on discord if you need more details on setting that up.

We do this kind of micro QA all the time when writing docs, which also helps you become a DVC expert little by little 🙂

Comment thread content/docs/command-reference/status.md
@imhardikj
Copy link
Copy Markdown
Contributor Author

@jorgeorpinel thank you for your review. I have updated the explanation with more details. Also made an attempt to include an example.

@shcheklein
Copy link
Copy Markdown
Contributor

@imhardikj thanks 🙏

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.

Yep, this is great. I just have some minor comments and suggestions for you. We can merge this after addressing those. Thanks @imhardikj!

Comment thread content/docs/command-reference/status.md Outdated
Comment thread content/docs/command-reference/status.md Outdated
Comment thread content/docs/command-reference/status.md Outdated
Comment thread content/docs/command-reference/status.md Outdated
Comment thread content/docs/command-reference/status.md Outdated
@jorgeorpinel
Copy link
Copy Markdown
Contributor

Thanks again @imhardikj

@jorgeorpinel jorgeorpinel merged commit 7fec065 into treeverse:master May 18, 2020
@imhardikj imhardikj deleted the dvc-status branch May 19, 2020 09:11
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.

cmd ref: describe update available output (for import stages) in status

4 participants