altsem
altsem
@Tmpod I pushed a commit to add this, fingers crossed it works. Ty for reporting it!
@rzikm Code looks good I think. Just some clippy complaints, "feat: " missing, tests.
@rzikm I've been busy with work recently, I'll try find some time soon to look at it.
~I haven't exactly grasped what's going on. pending_menu (and state of the arguments) should exist until the "log other" rev prompt is done and "log other" callback has been executed...
@rzikm Figured I'd share. After some digging I think I found an approach that might work. The idea is to: - Remove the `args: [&OsString]` parameter from prompt callbacks. -...
That's alright, no need to stress.
I continued on the branch: [altsem/value-arguments](https://github.com/altsem/gitu/tree/altsem/value-arguments) Made a test for grepping for something that doesn't exist. It seems that we get a crash when showing a screen with no lines....
I think we're ready. Good job! :) I did a rebase onto master, and will do a hack to add some entries to the changelog. Will make a release soon...
Pushed here with a rebase instead / some empty commits just for changelog entries purposes: https://github.com/altsem/gitu/pull/195
Could be the start of #58