nova-todo icon indicating copy to clipboard operation
nova-todo copied to clipboard

Add configuration option to change tag delimiter.

Open jasonplatts opened this issue 3 years ago • 2 comments

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

jasonplatts avatar Jul 28 '21 15:07 jasonplatts

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

evaneykelen avatar Mar 04 '22 20:03 evaneykelen

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.

jasonplatts avatar Mar 15 '22 14:03 jasonplatts