jsumners / alfred-emoji

Alfred workflow for searching and copying emoji
741 stars 32 forks source link

Simplify Script Filter #89

Closed vitorgalvao closed 1 year ago

vitorgalvao commented 1 year ago

There’s no need to assign to query, Alfred gets the whole input from $1.

jsumners commented 1 year ago

I am not ignoring your contributions. I just need some time to review and incorporate them.

vitorgalvao commented 1 year ago

If it makes it easier, I’m a member of the Alfred team (see the official org on GitHub for confirmation).

88 in particular makes this workflow ready for inclusion in the Gallery (blog; forum).

jsumners commented 1 year ago

If it makes it easier, I’m a member of the Alfred team

Awesome! Happy to have some attention from the team. As an aside, I'd love to see a more developer/git friendly way to generate the info.plist. Quite willing to discuss ideas in another issue or elsewhere.

Back on topic, this change works as described. I'm sure it wasn't clear to me when I started the project if all terms would be passed or not, hence the marshaling of them.