-
Notifications
You must be signed in to change notification settings - Fork 9
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
Interactive prompts #64
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Thx!
Can we get away without having a flag for interactive UI?
I.e.
from datalad.ui import ui
if ui.is_interactive:
....
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 |
9febdf4
to
c6095da
Compare
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
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 datalad/datalad#396 where this could be an awesome way to select to-be-edited items. |
Ci bumps: newer git annex and also python 3.8 not 3.7
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.