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

RF: Promote internal status() helper with docs, and breadth-first reporting #6006

Merged
merged 1 commit into from
Sep 26, 2021

Conversation

mih
Copy link
Member

@mih mih commented Sep 24, 2021

Matching the diff_dataset() helper.

I had this change lying around locally. it makes using status-functionality internally more convenient.

@mih mih added the semver-patch Increment the patch version when merged label Sep 24, 2021
@mih
Copy link
Member Author

mih commented Sep 24, 2021

I tried to convince the one mac runner on appveyor to actually do something but failed 5 times. The nature of the change makes it highly unlikely to fail only in this one particular run. Hence I'll ignore it.

@mih mih added the merge-if-ok OP considers this work done, and requests review/merge label Sep 24, 2021
@codecov
Copy link

codecov bot commented Sep 24, 2021

Codecov Report

Merging #6006 (a43d41a) into master (a50434e) will decrease coverage by 0.00%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6006      +/-   ##
==========================================
- Coverage   90.16%   90.16%   -0.01%     
==========================================
  Files         315      315              
  Lines       42188    42196       +8     
==========================================
+ Hits        38040    38047       +7     
- Misses       4148     4149       +1     
Impacted Files Coverage Δ
datalad/core/local/status.py 94.16% <75.00%> (-2.27%) ⬇️
datalad/support/tests/test_annexrepo.py 97.61% <0.00%> (+0.14%) ⬆️

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 a50434e...a43d41a. Read the comment docs.

@mih
Copy link
Member Author

mih commented Sep 26, 2021

OK, more docs should be more good

@mih mih merged commit 40ceaba into datalad:master Sep 26, 2021
@mih mih deleted the rf-status branch September 26, 2021 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-if-ok OP considers this work done, and requests review/merge semver-patch Increment the patch version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant