search

jasonplatts / nova-todo

A Panic Nova extension for todo and fixme management.
https://extensions.panic.com/extensions/jasonplatts/jasonplatts.TODO/
MIT License
29 stars 7 forks source link

Add configuration option to change tag delimiter. #27

Open jasonplatts opened 3 years ago

jasonplatts commented 3 years ago

Just a heads-up. I'm showing more TODO's than you are. I think the reason for this is that when you parse you actually check for 'TODO:' not just 'TODO' which is the format that I have traditionally used throughout my code. So I did a search and destroy on the local code in the project to add the colons, but I'd rather not have to do that to all my projects.

Is there any reason that you require the colon?? Dropping this requirement will obviously match more entries. I would venture to say that it's not really needed - 'TODO' should not return any false positives as it's pretty unambiguous.

Could I therefore suggest an enhancement; making it a delimiter that is user configurable via text box in the settings dialog? This way users can tailor it to their particular coding style (TODO- TODO> TODO... TODO-- (TODO) [TODO] etc/whatever). The semantics of some languages tend to favour different styles as well, not to mention that may coders will adopt the style of the code author if they are working on libraries extending existing code.

Cheers. And thanks for a great plugin.

/DM

Originally posted by @DeeEmm in https://github.com/jasonplatts/nova-todo/issues/26#issuecomment-888394473

evaneykelen commented 2 years ago

I'd be good to make the requirement for a trailing colon configurable. I work in codebases that use mixed styles (TODO and TODO:).

jasonplatts commented 2 years ago

Thanks Evan! It's helpful to know that others would also find this feature useful. I will aim to add this to an upcoming release.