-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Refactor test/remotes.py by merging inside remote classes #2878
Labels
enhancement
Enhances DVC
p3-nice-to-have
It should be done this or next sprint
refactoring
Factoring and re-factoring
Comments
I have started working on it. :) |
11 tasks
I haven't been able to continue working on this. I'll be able to continue on this only after Christmas. Sorry for not being quick on this. 🙂 |
@skshetry No worries 🙂 Thanks for the heads up! |
This was referenced Jan 4, 2020
This was referenced Jan 13, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
enhancement
Enhances DVC
p3-nice-to-have
It should be done this or next sprint
refactoring
Factoring and re-factoring
#2853 was just merged, which refactored different helper functions into
test/remotes.py
. There are different remote classes (such as S3, Azure, etc) and functions such asget_aws_url()
/get_gcp_url
.It'd be good to merge them inside their respective remote classes. :)
The text was updated successfully, but these errors were encountered: