Functions to remove all todo items that are done

[Nudin]

This adds functions that remove todo items that are marked as done.

• VimwikiRemoveDone/vimwiki#lst#remove_done removes them within the selected area.
• VimwikiRemoveDoneInList/vimwiki#lst#remove_done_in_list removes them in the list the cursor is in.
  • vimwiki#lst#remove_done_in_list has a parameter to define if it should work recursively. Default is not recursive.

Superseeding #391
Fixes #823 and others.

[Nudin]

The documentation still has to be improved. I wasn't sure if we should define both commands and whether default should be recursive or not. Recursive has the somewhat surprising but correct side effect that the status of items can decrease. See the test for example.

@tinmarino @rane-brown any thoughts?

[tinmarino]

I would put recursive as default: to get most work done which is harder to do by hand.

Very readble tests, really awesome.
Reading the last one, with spaces suprised me it was considered as a single list but better. Can you add tests for the behavior of 2 lists with content in between ?

* [X] Done 1
* [ ] Todo 1

This should break the list

* [ ] Todo Post space
* [X] Done Post space

[ubitux]

It probably doesn't belong in this PR, but what about multi-line TODO entries? I often use gq and ends up with multi-line entries, and I know they're not well supported in vimwiki (at least in markdown let g:vimwiki_hl_cb_checked = 1 will only gray-out the first line). So I'm curious how the removal of this PR behaves here in that situation.

[Nudin]

I would put recursive as default: to get most work done which is harder to do by hand.

Good argument, I changed it to recursive and updated the tests accordingly.

Reading the last one, with spaces suprised me it was considered as a single list but better. Can you add tests for the behavior of 2 lists with content in between ?

I actually found out about this behavior by your list_update.vader tests. ;) This PR doesn't change anything on the general behavior of list but reuses existing functions.

It probably doesn't belong in this PR, but what about multi-line TODO entries? I often use gq and ends up with multi-line entries, and I know they're not well supported in vimwiki (at least in markdown let g:vimwiki_hl_cb_checked = 1 will only gray-out the first line). So I'm curious how the removal of this PR behaves here in that situation.

Yes, better create a ticket for that (if it doesn't exist yet).

[tinmarino]

Perfect for me but I let you merge as you got this power