search

pimutils / todoman

✅ A simple, standards-based, cli todo (aka: task) manager.
https://todoman.readthedocs.io
ISC License
483 stars 78 forks source link

pass arguments properly to invoke #427

Closed bernieke closed 3 years ago

bernieke commented 3 years ago

The arguments being passed to invoke weren't doing anything.

Now they are properly parsed, converted (sort as list, due as integer, ...) and passed to invoke.

WhyNotHugo commented 3 years ago

Are you using click 8.0 by any change?

Click 8.0 was not supported and had this issue, but the latest merge to master fixed that.

WhyNotHugo commented 3 years ago

Oh, wait, I think I misunderstood this. Do you have an example usage that fails without this patch?

bernieke commented 3 years ago

I have following in my config.py:

default_command = "list --sort=due --due 168 --priority low --no-reverse tasks"

Without my changes the arguments to the list command don't have any effect.

bernieke commented 3 years ago

I had some trouble with the master->main switch, which caused the PR to get closed, and I couldn't figure out how to re-open it, so I made a new one: https://github.com/pimutils/todoman/pull/436