Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cucumber emoji is duplicated #133

Open
jufemaiz opened this issue Aug 8, 2018 · 5 comments
Open

Cucumber emoji is duplicated #133

jufemaiz opened this issue Aug 8, 2018 · 5 comments

Comments

@jufemaiz
Copy link
Contributor

jufemaiz commented Aug 8, 2018

There are currently two cucumber emoji:

Thoughts?

@keithpitt
Copy link
Member

Perhaps we should rename the custom one to cucumber_ruby or something? I remember when I first added the cucumber emoji, the Apple one didn't exist.

Even though cucumber currently resolves to the old one, I think it's an easy change for peeps to make if they want the old one back. And it's probably better to be consistent with actual emoji usage. What do you think @jufemaiz?

@jufemaiz
Copy link
Contributor Author

jufemaiz commented Aug 8, 2018

I'd probably go with your recommendation (happy to PR). I think it makes most sense (standard emoji as primary source » custom buildkite emoji as secondary source).

@ticky
Copy link
Contributor

ticky commented Aug 8, 2018

I’ve been wanting to address this, I actually recently implemented duplicate checking and this always comes up in its results!

We can’t rename the Unicode emoji, so renaming cucumber-the-tool would make sense, the trouble is that the custom emoji takes precedence, so renaming it means we retroactively change the existing emoji to the Unicode one, which is probably a bit weird.

I don’t think :cucumber_ruby: makes much sense as the tool supports about a dozen languages. Maybe :cucumber_specs:?

If you need a Unicode cucumber now, you can always just type a Unicode cucumber (🥒) and it’ll work fine! 👌🏼

@jufemaiz
Copy link
Contributor Author

jufemaiz commented Aug 8, 2018

@ticky it was your checker! Nice! Agree regarding :cucumber_specs: but this was more to highlight the name conflict for the BK team to work out how to resolve 😀👍

@ticky
Copy link
Contributor

ticky commented Aug 9, 2018

as the BK team’s resident emoji (a-little-too-)enthusiast, I think I’ve thus worked out how to resolve it 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants