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

fetch: introduce --tree-only #9140

Closed
wants to merge 1 commit into from
Closed

Conversation

efiop
Copy link
Contributor

@efiop efiop commented Mar 9, 2023

Very useful plumbing command to be able to restore full data index before actually downloading files.

This should be used as a step in regular fetch in the future instead of implicit cloud.pull() in output.get_dir_cache.

@dberenbaum
Copy link
Collaborator

If it's exposed in dvc fetch, what about --dir-only since it better matches the terminology on the user side with .dir files?

@efiop
Copy link
Contributor Author

efiop commented Mar 9, 2023

Thought about --dir-only, but that is leaking the internal object name into the outside, while the aim is to fetch the structure (trees) of your repo. --dir-only also makes sense if we go with #8884 and make our outputs use self.files first and not try to convert them into .dir object like we do right now. Taking a look at that last problem right now and might rename the flag here.

@efiop
Copy link
Contributor Author

efiop commented Jun 20, 2023

Closing for now, might revisit later

@efiop efiop closed this Jun 20, 2023
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