Skip to content

Feedback after release #662

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

Merged
merged 4 commits into from
Jun 8, 2021
Merged

Conversation

martindurant
Copy link
Member

@martindurant martindurant commented Jun 8, 2021

  • registry loose versions dropped

  • current directory cannot be deleted

  • local _strip_protocol to remove trailing "/"

Martin Durant added 3 commits June 8, 2021 11:48
- registry loose versions dropped

- current directory cannot be deleted
@isidentical
Copy link
Member

Apparently, some stuff is also broken on DVC related to the local filesystem, though didn't have time to debug it. Perhaps it was the info() with path-likes. Will look into it in depth tomorrow!

@martindurant
Copy link
Member Author

Thanks, @isidentical . Do you want to add DVC into the test runs here yet?

@isidentical
Copy link
Member

We have too many fs-unbound runs, so until they are all marked and grouped I don't think that it will worth it. CC: @efiop

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