docs: regular update (early August) #522

jorgeorpinel · 2019-08-06T23:51:19Z

No description provided.

…out docs For #487

per #509 (review)

static/docs/commands-reference/import-url.md

src/Documentation/glossary.js

static/docs/commands-reference/checkout.md

static/docs/commands-reference/commit.md

shcheklein

good stuff! just a couple of minor comments to address

per #522 (review)

per #532 (comment)

and reformat it in glossary.json with a ES6 template literal per #522 (comment)

per #522 (comment)

jorgeorpinel · 2019-08-09T22:32:00Z

src/Documentation/glossary.js

+project when [\`dvc init\`](/doc/commands-reference/init) is run, and
+[DVC-files](/doc/user-guide/dvc-file-format) are created in it. It\s typically
+also a Git repository.
+      `


If your question is whether the linter will fix lines that go over 80 chars, no. Neither with the previous or with this format does it fix them, neither with the hook nor run manually 🙁. It seems to leave strings alone.

Is that another issue to report to Aman?

we can def check if it's possible to configure it to fix them. In any case, I don't mind the format you suggested, its' def easier to support manually

cc @algomaster99 what do you think? (about pretty-quick being able to warp strings in js files)

looks a bit strange, but let's try

Applied new format to all glossary entries in 8781a71

static/docs/get-started/add-files.md

per #522 (comment)

…ular paragaph per #522 (review)

jorgeorpinel · 2019-08-10T00:43:13Z

I just thought that we are almost ready to merge this one.

I wish haha... I have lots of changes after reviewing the usage of the term "workspace" per a comment somewhere in this PR

BUT I just realized that should go in it's own PR! So I saved that work separately and just pushed changes for the pending comments above. Please check @shcheklein (:

I'll resolve conflicts now.

so diffs and file searches don't find this page when searching for them.

… tooltip

shcheklein

thanks 🎉 !

jorgeorpinel added 4 commits August 6, 2019 17:36

import-url: try and update example; review term "Get Started" through…

50212bd

…out docs For #487

update: change references to import stages being unlockable

c0cb7ed

per #509 (review)

cmd ref: standardize example headers and expandables up to dvc import

be18aad

cmd ref: standardize example headers and expandables up to dvc update

a388712

jorgeorpinel commented Aug 7, 2019

View reviewed changes

static/docs/commands-reference/import-url.md Outdated Show resolved Hide resolved

cmd ref: standardize example headers and expandables (finished)

6946023

jorgeorpinel changed the title ~~cmd ref: advanced examples for import and update; et al [WIP]~~ docs: regular update (late July) Aug 7, 2019

jorgeorpinel changed the title ~~docs: regular update (late July)~~ docs: regular update (early August) Aug 7, 2019

jorgeorpinel requested a review from shcheklein August 7, 2019 05:22

shcheklein reviewed Aug 7, 2019

View reviewed changes

src/Documentation/glossary.js Outdated Show resolved Hide resolved

shcheklein reviewed Aug 7, 2019

View reviewed changes

static/docs/commands-reference/checkout.md Outdated Show resolved Hide resolved

shcheklein reviewed Aug 7, 2019

View reviewed changes

static/docs/commands-reference/commit.md Outdated Show resolved Hide resolved

shcheklein requested changes Aug 7, 2019

View reviewed changes

jorgeorpinel added 5 commits August 8, 2019 00:53

term: update "workspace" glossary entry title and desc

e641349

per #522 (review)

cmd ref: updated shared example intro in checkout, commit, and fetch

f1dc617

per #522 (review)

lint: Make YAML blocks valid

2536b99

per #532 (comment)

term: update "workspace" glossary entry and

686561b

and reformat it in glossary.json with a ES6 template literal per #522 (comment)

get-started: updates for more and better use of term "workspace"

fc28eb0

per #522 (comment)

jorgeorpinel commented Aug 9, 2019

View reviewed changes

This comment has been minimized.

Sign in to view

shcheklein reviewed Aug 9, 2019

View reviewed changes

static/docs/get-started/add-files.md Outdated Show resolved Hide resolved

This comment has been minimized.

Sign in to view

jorgeorpinel added 2 commits August 9, 2019 19:35

glossary: use new format with ES6 template literals

8781a71

per #522 (comment)

get-started: make noted text about git commit in add-files into a reg…

328b2ec

…ular paragaph per #522 (review)

jorgeorpinel added 4 commits August 10, 2019 00:03

pipeline: small improvement in cmd ref short desc.

b949520

Merge remote-tracking branch 'upstream/master'

796e799

tutorial: change <<<<<<< etc Git conflict markers to HTML symbols

ee1cf35

so diffs and file searches don't find this page when searching for them.

term: Review "cache directory" usage and add <abbr> tags for glossary…

6d1f6c1

… tooltip

shcheklein approved these changes Aug 10, 2019

View reviewed changes

shcheklein merged commit f9b9e33 into iterative:master Aug 10, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs: regular update (early August) #522

docs: regular update (early August) #522

jorgeorpinel commented Aug 6, 2019 •

edited

Loading

shcheklein left a comment

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as outdated.

This comment was marked as resolved.

jorgeorpinel Aug 9, 2019 •

edited

Loading

shcheklein Aug 9, 2019

jorgeorpinel Aug 9, 2019

jorgeorpinel Aug 10, 2019

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

jorgeorpinel commented Aug 10, 2019 •

edited

Loading

shcheklein left a comment

docs: regular update (early August) #522

docs: regular update (early August) #522

Conversation

jorgeorpinel commented Aug 6, 2019 • edited Loading

shcheklein left a comment

Choose a reason for hiding this comment

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as outdated.

This comment was marked as resolved.

jorgeorpinel Aug 9, 2019 • edited Loading

Choose a reason for hiding this comment

shcheklein Aug 9, 2019

Choose a reason for hiding this comment

jorgeorpinel Aug 9, 2019

Choose a reason for hiding this comment

jorgeorpinel Aug 10, 2019

Choose a reason for hiding this comment

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

jorgeorpinel commented Aug 10, 2019 • edited Loading

shcheklein left a comment

Choose a reason for hiding this comment

jorgeorpinel commented Aug 6, 2019 •

edited

Loading

jorgeorpinel Aug 9, 2019 •

edited

Loading

jorgeorpinel commented Aug 10, 2019 •

edited

Loading