Gitmoji: Performance improve, merging and reverting

Created on 16 Nov 2016  ยท  5Comments  ยท  Source: carloscuesta/gitmoji

Hello @carloscuesta :sunglasses:!


Great project, I use most of them myself, in my personal projects I usually don't leave a single commit without starting the message with a contextual emoji, it makes the log easier to read and to understand. Just wanted to share some that I use that could be helpful:


  • Emoji: :racehorse:
  • Code: :racehorse:
  • Description: Performance improvements

  • Emoji: :cactus:
  • Code: :cactus:
  • Description: Merge commits

  • Emoji: :back:
  • Code: :back:
  • Description: Revert commits
emoji

All 5 comments

Hey @jonathanwiesel

That's a great issue! Thanks for taking the time to fill this one ๐Ÿ‘

Great ideas in there, personally I prefer to use :zap: instead of ๐ŸŽ for performance, the reverting commits emoji looks good for me and the cactus one it's cool because there are like "two branches", but could be a little bit confusing though.

The revert one is nice, but I think โช (:rewind:) might be prettier and more coherent with existing ones (up-/downgrading dependencies).

As for merging icon, it's not clear at the first glance what the ๐ŸŒต (:cactus:) means, so I would rather propose ๐Ÿ”€ (:twisted_rightwards_arrows:) - it's not perfect, but it suggests interaction of two branches, it's coherent and for merging upstream branch into downstream one it's actually pretty accurate. Although it's pretty inconvenient to write, it shouldn't be an issue once the CLI tool (https://github.com/carloscuesta/gitmoji-cli) is in place.

Let me know what you think. If these are fine I will gladly submit a PR. ๐Ÿ™‚

Hey @virzen feel free to submit a pull request for the rewind and merge ones ๐Ÿ˜„

Great @virzen !

Was this page helpful?
0 / 5 - 0 ratings

Related issues

exodusanto picture exodusanto  ยท  3Comments

Taranys picture Taranys  ยท  3Comments

SirWindfield picture SirWindfield  ยท  4Comments

rafmst picture rafmst  ยท  4Comments

jpventura picture jpventura  ยท  3Comments