todotxt / todo.txt-cli

☑️ A simple and extensible shell script for managing your todo.txt file.
http://todotxt.org
GNU General Public License v3.0
5.55k stars 712 forks source link

Prioritizing multiple items at once does not match documentation #413

Closed har96 closed 1 year ago

har96 commented 1 year ago

Do you want to request a feature or report a bug? bug

What is the current behavior?

$ todo.sh pri 1 a, 2 b, 3 c
usage: todo.sh pri ITEM# PRIORITY[, ITEM# PRIORITY, ...]
note: PRIORITY must be anywhere from A to Z.

Which versions todo.sh are you using?

Run todo.sh -V

TODO.TXT Command Line Interface v@DEV_VERSION@

Homepage: http://todotxt.org
Code repository: https://github.com/todotxt/todo.txt-cli/
Contributors: https://github.com/todotxt/todo.txt-cli/graphs/contributors
License: https://github.com/todotxt/todo.txt-cli/blob/master/LICENSE

Which Operating System are you using? Ubuntu

Which version of bash are you using?

Run bash --version

zsh 5.9 (x86_64-ubuntu-linux-gnu)

Description

The documentation for the pri command does not match its behavior. Documentation leads you to believe that you should place commas between <item> <priority> pairs, when in reality the code fails if a comma is placed next to the priority. Without commas, the command works as expected:

$ todo.sh pri 1 a 2 b 3 c
inkarkat commented 1 year ago

Duplicate of #401. The confusing syntax affects almost all built-in commands, not just pri.