Skip to content

dvc: introduce DvcTree#3639

Merged
efiop merged 1 commit into
treeverse:masterfrom
efiop:import_checkout
Apr 15, 2020
Merged

dvc: introduce DvcTree#3639
efiop merged 1 commit into
treeverse:masterfrom
efiop:import_checkout

Conversation

@efiop
Copy link
Copy Markdown
Contributor

@efiop efiop commented Apr 15, 2020

This is useful as it is in dvc ls implementation, but this is also a
prerequisite for RepoTree that will combine git(workspace) files and
dvc files into one tree, allowing things like dvc metrics and
external_repo to have a universal interface to access whatever
file/dir they want without cumbersome git/dvc path detection.

  • ❗ I have followed the Contributing to DVC checklist.

  • 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here. If the CLI API is changed, I have updated tab completion scripts.

  • ❌ I will check DeepSource, CodeClimate, and other sanity checks below. (We consider them recommendatory and don't expect everything to be addressed. Please fix things that actually improve code or fix bugs.)

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

This is useful as it is in `dvc ls` implementation, but this is also a
prerequisite for `RepoTree` that will combine git(workspace) files and
dvc files into one tree, allowing things like `dvc metrics` and
`external_repo` to have a universal interface to access whatever
file/dir they want without cumbersome git/dvc path detection.
@efiop efiop changed the title dvc: introduce DvcTree [WIP] dvc: introduce DvcTree Apr 15, 2020
@efiop efiop changed the title [WIP] dvc: introduce DvcTree dvc: introduce DvcTree Apr 15, 2020
@efiop efiop merged commit c60f364 into treeverse:master Apr 15, 2020
@efiop efiop deleted the import_checkout branch April 21, 2020 13:36
@efiop efiop restored the import_checkout branch April 21, 2020 13:36
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.

1 participant