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

cmd ref: -R flag for update command #1259

Closed
nik123 opened this issue May 7, 2020 · 6 comments · Fixed by #1271
Closed

cmd ref: -R flag for update command #1259

nik123 opened this issue May 7, 2020 · 6 comments · Fixed by #1271
Assignees
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: ref Content of /doc/*-reference type: enhancement Something is not clear, small updates, improvement suggestions

Comments

@nik123
Copy link
Contributor

nik123 commented May 7, 2020

Command reference for update command should contain -R/--recursive flag.

See issue iterative/dvc#3511 and PR iterative/dvc#3760 in the core repo.

@jorgeorpinel
Copy link
Contributor

Thanks. It will be updated if/when the flag is implemented and released. Currently it doesn't exist in the command. Best

@shcheklein
Copy link
Member

@jorgeorpinel not sure why did we close this? iterative/dvc#3760 there is core repo PR @nik123 has prepared. So, we indeed need a ticket.

@shcheklein shcheklein reopened this May 7, 2020
@shcheklein shcheklein added A: docs Area: user documentation (gatsby-theme-iterative) type: enhancement Something is not clear, small updates, improvement suggestions labels May 7, 2020
@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented May 7, 2020

I didn't realize there was a PR for this and thought nik was a user trying to report a bug 😅

Updating the docs is a checkbox in that other PR too, but yeah sure, doesn't hurt to have this issue.

@shcheklein
Copy link
Member

@jorgeorpinel that checkbox says " ... or issue, at least ..." :)

@nik123
Copy link
Contributor Author

nik123 commented May 8, 2020

@jorgeorpinel sorry, it may not have been clear that there is PR for this. I've updated description with link to original PR.

@jorgeorpinel
Copy link
Contributor

Thanks!

efiop added a commit that referenced this issue May 8, 2020
@efiop efiop self-assigned this May 8, 2020
@iesahin iesahin added the C: ref Content of /doc/*-reference label Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: ref Content of /doc/*-reference type: enhancement Something is not clear, small updates, improvement suggestions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants