search

Chaosthebot / Chaos

A social coding experiment that updates its own code democratically.
http://chaosthebot.com
MIT License
2.44k stars 210 forks source link

Use symbol emojis instead of human emojis #472

Closed PlasmaPower closed 7 years ago

PlasmaPower commented 7 years ago

When I'm glancing through emails, I have to look closely to see what :ok_woman: and :no_good: are. It'd be a lot easier with :no_entry: and :white_check_mark:. The different colors make it a lot clearer.

rhengles commented 7 years ago

Personally, I prefer more 🆗 or ✅ than 🎉 .

PlasmaPower commented 7 years ago

@rhengles :white_check_mark: looks like a good choice. Changed.

Thoughts on the stale PR with merge conflicts emoji? It was :no_good:. I changed it to :broken_heart: as it's a bit different than :no_entry:. I'm reconsidering that though. Which does everyone else prefer?

PlasmaPower commented 7 years ago

@andrewda Re-review needed after change

mark-i-m commented 7 years ago

@PlasmaPower Have you just been stockpiling PRs?

PlasmaPower commented 7 years ago

@mark-i-m I stockpile ideas, then sometimes I feel like doing stuff.

PlasmaPower commented 7 years ago

I have chaosbot's full attention - literally all the PRs it is processing right now are mine :smile:.

mark-i-m commented 7 years ago

Shameless plug: #420

PlasmaPower commented 7 years ago

@mark-i-m yeah I'm really appreciating that right now.

PlasmaPower commented 7 years ago

Now includes stalled CI failed message (recently merged).

chaosbot commented 7 years ago

:ok_woman: PR passed with a vote of 10 for and 0 against, a weighted total of 10.0 and a threshold of 6.5, and a current meritocracy review.

See merge-commit 2970afce43b460f35785fcb285382a3c293b4ff5 for more details.