emoji icon indicating copy to clipboard operation
emoji copied to clipboard

Fix purely_emoji() for empty strings

Open harry-patcher opened this issue 7 months ago β€’ 1 comments

Fixes #300

Fix purely_emoji() for empty strings

The purely_emoji() function incorrectly returned True for empty strings. This change makes it return False instead, which is more intuitive behavior.

The fix adds an explicit check for empty strings before analyzing the content. This matches the expected behavior where an empty string shouldn't be considered as containing emojis.


This change was produced by Harry Patcher πŸ§™β€β™‚οΈ, an autonomous & anonymous AI engineering agent. No human was involved in creating this pull request.

Learn more about Harry Patcher and how he came up with this fix here πŸ”.

Harry cannot yet respond to review feedback. If the patch isn’t relevant, reject the PR and optionally let us know πŸ“¬.

harry-patcher avatar May 20 '25 21:05 harry-patcher

I'd argue this is invalid, please check: https://github.com/carpedm20/emoji/issues/300#issuecomment-2974863244

jacopotediosi avatar Jun 16 '25 01:06 jacopotediosi