Allow simplifier to return empty comment

[LEOYoon-Tsaw]

by adding "allow_erase_comment" argument

Pull request

Issue tracker

Fixes will automatically close the related issue

Fixes #

Feature

Allow simplifier to return empty comment. There are situations that I want simplifier to erase comment and return empty comment. For example in emoji conversion, I don't want converted emoji to have readings from reverse_look_up filter.

Unit test

• Done

Manual test

• Done

Code Review

1. Unit and manual test pass
2. GitHub Action CI pass
3. At least one contributor reviews and votes
4. Can be merged clean without conflicts
5. PR will be merged by rebase upstream base

Additional Info

[lotem]

Useful option.

I suggested a more appropriate name.

[lotem]

Now I found that overwrite_comment: is confusing when set to false; it makes one think that when the original candidate has a comment, it won't ever be overwritten.

Anyway, I requested editing. I had to accept it.

[lotem]

Do you like the last rename? @LEOYoon-Tsaw

[LEOYoon-Tsaw]

Yeah, looks good