flashtext icon indicating copy to clipboard operation
flashtext copied to clipboard

Consider droping check for container type in add_keywords_from_list

Open GrazingScientist opened this issue 2 years ago • 0 comments

Although the name of the function clearly states that it expects a list, add_keywords_from_list should take any container.

In the implementation, the only thing done is to loop over the given list. However, this would also be possible with a set with no change in behavior. Is there a reason why it has to be a list?

GrazingScientist avatar Dec 06 '21 06:12 GrazingScientist