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.

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

docs: regular update (late June) #456

Merged
merged 10 commits into from
Jun 27, 2019
Merged

docs: regular update (late June) #456

merged 10 commits into from
Jun 27, 2019

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Jun 25, 2019

Misc. needed updates/improvements to several docs.

@jorgeorpinel jorgeorpinel changed the title docs: regular update (late June) docs: regular update (late June) [WIP] Jun 25, 2019
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.

just a few minor comments. This a great update, @jorgeorpinel ! 🎉

@shcheklein

This comment has been minimized.

@jorgeorpinel

This comment has been minimized.

@shcheklein
Copy link
Member

@jorgeorpinel np! it's not a high priority. Let's prioritize now "dvc import/dvc get" - it's super important :)

@jorgeorpinel
Copy link
Contributor Author

Got it. And probably pkg commands too?

p.s. can this PR be merged? 😬

@shcheklein
Copy link
Member

@jorgeorpinel sure! there are no pkg commands anymore, that what I meant - dvc import, dvc get, dvc get-url

also, we will need to document DVC API after that - dvc.api.open ... dvc.api.get_content

please, let's remove WIP from the PR :)

@shcheklein shcheklein merged commit 85d6684 into iterative:master Jun 27, 2019
@shcheklein
Copy link
Member

cool stuff @jorgeorpinel 🎉

@jorgeorpinel
Copy link
Contributor Author

Understood!

@jorgeorpinel jorgeorpinel changed the title docs: regular update (late June) [WIP] [WIP] Aug 7, 2019
@jorgeorpinel jorgeorpinel changed the title [WIP] docs: regular update (late June) Aug 7, 2019
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.

2 participants