Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Write import, get, and get-url refs; std. existing refs; other doc format/lang. updates #464

Merged
merged 45 commits into from
Jul 15, 2019
Merged

Write import, get, and get-url refs; std. existing refs; other doc format/lang. updates #464

merged 45 commits into from
Jul 15, 2019

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Jun 29, 2019 •
edited
Loading

Fix #385 based on desc in iterative/dvc#2160

@jorgeorpinel jorgeorpinel mentioned this pull request Jun 29, 2019
Merged
2 tasks
@jorgeorpinel jorgeorpinel requested a review from shcheklein July 1, 2019 18:48
@jorgeorpinel jorgeorpinel changed the title cmd ref: doc import, get, and get-url [WIP] cmd ref: doc import, get, and get-url; and other misc. updates Jul 1, 2019
@jorgeorpinel
Copy link
Contributor Author

This is still WIP @shcheklein but you may want to review the changes from term: ... commits. Those are just misc. doc improvements that affect several files. Sorry its all mixed up in this PR...

@shcheklein shcheklein mentioned this pull request Jul 1, 2019
Closed
shcheklein
shcheklein requested changes Jul 2, 2019
View reviewed changes
Copy link
Member

@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.

please, check the comments

shcheklein
shcheklein requested changes Jul 3, 2019
View reviewed changes
Copy link
Member

@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.

good progress 👍 , check the few comments I left

This was referenced Jul 8, 2019
Merged
Merged
@jorgeorpinel jorgeorpinel changed the title [WIP] cmd ref: doc import, get, and get-url; and other misc. updates cmd ref: doc import, get, and get-url; other misc. updates Jul 9, 2019
@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jul 9, 2019 •
edited
Loading

Finally removing WIP from this PR title as first versions of dvc import and dvc get are now included in the changes. A few questions (in HTML comments) and examples pending.

@jorgeorpinel
guide: remove unnecessary sentence from share-data
23ca5db 
Per https://github.com/iterative/dvc.org/pull/472/files#r301482051
@jorgeorpinel
status: rewrap usage code block
880736d 
Per #416 (review)
@jorgeorpinel
cases: mention directories and dvc run in data-and-model-files-vers…
3fb8da1 
…ioning

Related to #431 (comment)
@jorgeorpinel
cmd ref: First version of import and get, with updated -url cou…
b61b26d 
…nterparts

For #385
@jorgeorpinel
Simplify notes about single-use commands.
b94c799 
Per #464 (review)
@jorgeorpinel
cmd ref: Add "Git server e.g. Github" note to import and get summ…
88dddaf 
…aries

Per #464 (review)
@jorgeorpinel
cmd ref: updated url arg desc in import and get
ae0ed5c 
Per #464 (review)
@jorgeorpinel
cmd ref: add note abot http and ssh protocols to get and import
65980d9 
Per #464 (review)
@jorgeorpinel
init: clarify "local" (repo) term
aa4067b 
Per #464 (review)
@jorgeorpinel
remote: fix grammar in --local option of modify and remove
6780754 
Per #464 (review)
@jorgeorpinel
term: review use of "config(uration) file" and link to /doc/commands-…
7f33a3a 
…reference/config

Related to #464 (review)
@jorgeorpinel
remote: std --local opt desc
d5b38f4 
Per #464 (review)
@jorgeorpinel
cmd ref: remove outdated comment from get and import
ce2dc0c
@jorgeorpinel
cmd ref: make note about single-use commands into regular paragraphs
dab6a36 
Per #464 (review)
@jorgeorpinel
cmd ref: udpate url arg desc in get and import (again)
4f9308f 
Per #464 (review)
@jorgeorpinel
version: remove unnecessary note
92a5238 
Per #464 (review)
@jorgeorpinel
s3: update info on boto3 methods and permissions required...
fd7f428 
in `imoprt-url`, `get-url` as well as `remote` and `remote add` command refs.
Updates also related guides (install and config).
@jorgeorpinel
init: update with details about using or nto a Git repo for the DVC p…
13bde23 
…roject

Per #464 (review)
@jorgeorpinel
cmd ref: improve desc of import and get commands, et al
c38cc8e 
Per #464 (review)

Includes other misc changes.
@jorgeorpinel
cmd ref: fix command to install DVC with pip inc all remotes
377410c 
Per 9d0819d#r34289598
@jorgeorpinel
install: add [oss] to list of optional deps when installing via pip
6d0d0e0 
Per #464 (review)
@jorgeorpinel
cmd ref: be more specific about what import and get are for...
2b1008a 
in import-url and get-url notes about them

Per #464
@jorgeorpinel
cmd ref: clarify that get and get-url download files anywhere...
d53434d 
not just into DVC projects.

Per #464 (review)
@jorgeorpinel
Copy link
Contributor Author

Merging is blocked for me @shcheklein . You have to accept the changes, please check 🙂

shcheklein
shcheklein approved these changes Jul 15, 2019
View reviewed changes
Copy link
Member

@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.

Great stuff! 🎉

@shcheklein shcheklein merged commit ad20d2e into iterative:master Jul 15, 2019
@jorgeorpinel
Copy link
Contributor Author

Yay finally haha 🦉

@jorgeorpinel jorgeorpinel changed the title cmd ref: doc import, get, and get-url; other misc. updates Write import, get, and get-url refs; standardize existing refs; other misc. doc format/lang. updates Mar 16, 2023
@jorgeorpinel jorgeorpinel changed the title Write import, get, and get-url refs; standardize existing refs; other misc. doc format/lang. updates Write import, get, and get-url refs; std. existing refs; other doc format/lang. updates Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shcheklein shcheklein shcheklein approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

document dvc import and dvc get (previously pkg commands)
2 participants
@jorgeorpinel @shcheklein