Skip to content
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

Correct the docstring for BasePath.parents #936

Merged
merged 1 commit into from
Mar 21, 2022
Merged

Conversation

jwodder
Copy link
Member

@jwodder jwodder commented Mar 14, 2022

No description provided.

@jwodder jwodder added the documentation Changes only affect the documentation label Mar 14, 2022
@codecov
Copy link

codecov bot commented Mar 14, 2022

Codecov Report

Merging #936 (0497a7b) into master (a1a7fd5) will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #936      +/-   ##
==========================================
+ Coverage   87.31%   87.36%   +0.05%     
==========================================
  Files          62       62              
  Lines        7662     7695      +33     
==========================================
+ Hits         6690     6723      +33     
  Misses        972      972              
Flag Coverage Δ
unittests 87.36% <ø> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
dandi/misctypes.py 61.68% <ø> (ø)
dandi/tests/test_files.py 100.00% <0.00%> (ø)
dandi/pynwb_utils.py 83.85% <0.00%> (+0.07%) ⬆️
dandi/files.py 79.44% <0.00%> (+0.07%) ⬆️
dandi/tests/fixtures.py 97.89% <0.00%> (+0.09%) ⬆️
dandi/organize.py 82.32% <0.00%> (+0.13%) ⬆️
dandi/tests/test_organize.py 95.00% <0.00%> (+0.33%) ⬆️
dandi/support/threaded_walk.py 94.64% <0.00%> (+1.78%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1a7fd5...0497a7b. Read the comment docs.

@@ -143,7 +143,7 @@ def parent(self: P) -> P:
def parents(self: P) -> Tuple[P, ...]:
"""
A tuple of the path's ancestors, starting at the parent and going up to
(but not including) the root of the hierarchy
(and including) the root of the hierarchy
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

didn't test -- will just believe you ;)

@yarikoptic yarikoptic merged commit d0a7aac into master Mar 21, 2022
@yarikoptic yarikoptic deleted the fix-parents-docs branch March 21, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes only affect the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants