Code style and documentation fixes as per yoh review

Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
dandi · Feb 25, 2023 · 5eae970 · 5eae970
1 parent 3ebf99f
commit 5eae970
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/dandi/files/__init__.py b/dandi/files/__init__.py
@@ -105,8 +105,7 @@ def find_dandi_files(
                 raise ValueError(
                     "Path {str(p)!r} is not inside Dandiset path {str(dandiset_path)!r}"
                 )
-        bidsdd = None
-        path_queue.append((Path(p), bidsdd))
+        path_queue.append((Path(p), None))
     bids_roots = []
     while path_queue:
         p, bidsdd = path_queue.popleft()

diff --git a/dandi/tests/test_validate.py b/dandi/tests/test_validate.py
@@ -28,12 +28,12 @@ def test_validate_bids(bids_examples, tmp_path, dataset):
 
 def test_validate_bids_onefile(bids_error_examples, tmp_path):
     """
-    Dedicated test using to single-file validation.
+    Dedicated test using single-file validation.
 
     Notes
     -----
-    * Due to the dataset-wide scope of BIDS, issues have arisen and can potentially arise again
-    with single-file handling. Best to keep this in to always make sure.
+    * Due to the dataset-wide scope of BIDS, issues with single-file handling have arisen and can
+    potentially arise again. Best to keep this in to always make sure.
     * This can be further automated thanks to the upstream `.ERRORS.json` convention to be
     performed on all error datasets, but that might be overkill since we test the datasets as
     a whole anyway.