search

preslavmihaylov / todocheck

A static code analyser for annotated TODO comments
MIT License
426 stars 42 forks source link

Add support for case insensitive TODOs #167

Closed Emoto13 closed 3 years ago

Emoto13 commented 3 years ago

Support for case insensitive todos as per #48

Emoto13 commented 3 years ago

@preslavmihaylov Ready for review

preslavmihaylov commented 3 years ago

Overall, very good approach. I like that you've used something similar to a decorator to achieve this as this makes it quite flexible when adding new todo matchers.

After resolving some of the minor issues I mentioned, we're good to go to merge this.

Emoto13 commented 3 years ago

@preslavmihaylov Ready for review