Open adswa opened 2 years ago
Will do! However, I will delay working on this a bit until a query command has been introduced - maybe the need for interactive prompts doesn't exist anymore with the new command or api change
Merging #64 (c6095da) into master (61f760b) will decrease coverage by
2.62%
. The diff coverage is2.22%
.
@@ Coverage Diff @@
## master #64 +/- ##
==========================================
- Coverage 82.33% 79.71% -2.63%
==========================================
Files 10 11 +1
Lines 334 419 +85
==========================================
+ Hits 275 334 +59
- Misses 59 85 +26
Impacted Files | Coverage Δ | |
---|---|---|
datalad_xnat/init.py | 70.00% <0.00%> (-2.59%) |
:arrow_down: |
datalad_xnat/update.py | 73.07% <3.12%> (-12.00%) |
:arrow_down: |
datalad_xnat/tests/test_init.py | 100.00% <0.00%> (ø) |
|
datalad_xnat/parser.py | 88.88% <0.00%> (+1.93%) |
:arrow_up: |
datalad_xnat/platform.py | 79.78% <0.00%> (+3.19%) |
:arrow_up: |
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 61f760b...c6095da. Read the comment docs.
Here's an updated Gif of how this feature looks like:
shall I port this into the current state of the code?
I am putting this PR in draft mode for now. It would need to be revived and conflicts removed.
Also pinging https://github.com/datalad/datalad/issues/396 where this could be an awesome way to select to-be-edited items.
Here is a very quick and dirty implementation of interactive prompts. I think I will try to intergrate further developments of this in conjunction with a query command. See the comment with the GIF below to see how this feature behaves.