Skip to content

repofs: base on fsspec#7581

Merged
efiop merged 8 commits into
treeverse:mainfrom
efiop:repofs
Apr 18, 2022
Merged

repofs: base on fsspec#7581
efiop merged 8 commits into
treeverse:mainfrom
efiop:repofs

Conversation

@efiop
Copy link
Copy Markdown
Contributor

@efiop efiop commented Apr 17, 2022

Initial migration to fsspec.

repofs is meant to only accept correct posix paths relative to the root of the repo, but I've left some logic to allow for abspath and ntpath support to have a bit more flexibility for users.

Future improvements:

  • split repofs into overlayfs(fs + dvc) and subrepofs
  • cleanup __init__ logic

Fixes #7384

@efiop efiop requested a review from a team as a code owner April 17, 2022 22:35
@efiop efiop requested a review from pmrowla April 17, 2022 22:35
@efiop efiop added the refactoring Factoring and re-factoring label Apr 17, 2022
@efiop efiop requested review from skshetry and removed request for pmrowla April 17, 2022 22:35
@efiop efiop changed the title repofs: base on fsspec [WIP] repofs: base on fsspec Apr 17, 2022
@efiop efiop changed the title [WIP] repofs: base on fsspec repofs: base on fsspec Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Factoring and re-factoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

repofs: use repo paths

1 participant