Dvc: Refactor test/remotes.py by merging inside remote classes

Created on 1 Dec 2019  路  3Comments  路  Source: iterative/dvc

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 as get_aws_url()/get_gcp_url.

It'd be good to merge them inside their respective remote classes. :)

enhancement p3-nice-to-have refactoring
Source
picture skshetry

Most helpful comment

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. :slightly_smiling_face:

picture skshetry on 17 Dec 2019
👍2

All 3 comments

I have started working on it. :)

skshetry picture skshetry on 1 Dec 2019
❤2

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. :slightly_smiling_face:

skshetry picture skshetry on 17 Dec 2019
👍2

@skshetry No worries 馃檪 Thanks for the heads up!

efiop picture efiop on 17 Dec 2019
Was this page helpful?
0 / 5 - 0 ratings

Related issues

drorata picture drorata  路  46Comments
luchoPipe87 picture luchoPipe87  路  69Comments
shcheklein picture shcheklein  路  36Comments
andrethrill picture andrethrill  路  70Comments
dmpetrov picture dmpetrov  路  35Comments