Stone_Red/dvc
1
0
Fork 0
mirror of https://github.com/Stone-Red-Code/dvc.git synced 2026-11-02 07:18:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

ls-url: make config and recursive kwarg-only

Browse Source
This commit is contained in:
Ronan Lamy
2022-10-04 14:37:29 +03:00
committed by Ruslan Kuprieiev
parent d60edcd88e
commit a3f20eb6f4
2 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
dvc/repo/ls_url.py
+1 -1
View File
@@ -10,7 +10,7 @@ def parse_external_url(url, config=None):
return fs, fs_path
def ls_url(url, config=None, recursive=False):
def ls_url(url, *, config=None, recursive=False):
fs, fs_path = parse_external_url(url, config=config)
try:
info = fs.info(fs_path)
dvc/testing/workspace_tests.py
+6 -4
View File
@@ -154,7 +154,7 @@ class TestLsUrl:
def test_file(self, cloud, fname):
cloud.gen({fname: "foo contents"})
fs, fs_path = parse_external_url(cloud.url, cloud.config)
result = ls_url(str(cloud / fname), cloud.config)
result = ls_url(str(cloud / fname), config=cloud.config)
match_files(
fs,
result,
@@ -166,7 +166,7 @@ class TestLsUrl:
{"dir/foo": "foo contents", "dir/subdir/bar": "bar contents"}
)
fs, _ = parse_external_url(cloud.url, cloud.config)
result = ls_url(str(cloud / "dir"), cloud.config)
result = ls_url(str(cloud / "dir"), config=cloud.config)
match_files(
fs,
result,
@@ -181,7 +181,9 @@ class TestLsUrl:
{"dir/foo": "foo contents", "dir/subdir/bar": "bar contents"}
)
fs, _ = parse_external_url(cloud.url, cloud.config)
result = ls_url(str(cloud / "dir"), cloud.config, recursive=True)
result = ls_url(
str(cloud / "dir"), config=cloud.config, recursive=True
)
match_files(
fs,
result,
@@ -193,4 +195,4 @@ class TestLsUrl:
def test_nonexistent(self, cloud):
with pytest.raises(URLMissingError):
ls_url(str(cloud / "dir"), cloud.config)
ls_url(str(cloud / "dir"), config=cloud.config)