Skip to content
This repository has been archived by the owner on Jul 5, 2022. It is now read-only.

Corrections in #27 applied. Titles changed back to "Step n" #33

Merged
merged 1 commit into from
Mar 8, 2021

Conversation

iesahin
Copy link
Contributor

@iesahin iesahin commented Mar 8, 2021

This fixes #27. Thank you @jorgeorpinel

@iesahin iesahin merged commit 8cb83c3 into master Mar 8, 2021
Comment on lines -9 to 10
"title": "Track a file or directory",
"title": "Step 1",
"text": "01-track-a-file-or-directory.md"
Copy link
Contributor

Choose a reason for hiding this comment

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

Alternatively we could keep the full titles here and remove the # (H1) titles from .md files.

Copy link
Contributor

Choose a reason for hiding this comment

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

I guess that how it was before this change. OK, as you prefer 👍

Comment on lines +9 to +10
> `data.xml`. Let's check whether the hash calculated by `md5sum` is identical
> with the hash calculated by DVC.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's check whether the hash calculated by md5sum is identical with the hash calculated by DVC.

Oops, this part was OK as a regular paragraph.

Comment on lines +3 to +12
The command `dvc status` can compare the data on the workspace with the data saved
on the cache.

to compare files between the data stored on cache and the default remote.
`dvc status --cloud`{{execute}}

Without the `-c, --cloud` option

`dvc status`{{execute}}

checks whether workspace files are up to date with _local cache._
checks whether workspace files are up to date with the _local cache._
Copy link
Contributor

Choose a reason for hiding this comment

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

This order is still confusing IMO. Why not use plain status first and -c after?

Copy link
Contributor

Choose a reason for hiding this comment

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

Or just mention plain status first (without executable command) and go on to using status -c which is what matters at this point. Plain status is used later.

@jorgeorpinel
Copy link
Contributor

Thank you @iesahin it feels more streamlined now for sure 🙏

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GS/Versioning copy edits (1.0)
2 participants