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

✨ Set default emoji format to emoji #1248

Merged
merged 1 commit into from
Jan 28, 2024
Merged

✨ Set default emoji format to emoji #1248

merged 1 commit into from
Jan 28, 2024

Conversation

mikelorant
Copy link
Contributor

@mikelorant mikelorant commented Jan 28, 2024

The default emoji format was originally set to code for the following reason:

The reason behind this was to be able to use the shortcode to favour
the readability on certain terminals that do not support Emojis!
@carloscuesta

Terminals are considerably more advanced on all platforms and the standard Gitmoji set is simple enough that all terminals would display them correctly nowadays.

This change updates the default format from code to emoji.

The benefits of this change:

  • Improve display of emojis in commit titles, pull request titles and list of pull requests on some remote repository providers.

  • Removes need to modify the pager to allow git log to display emojis.

Fixes: #1247

The default emoji format was originally set to `code` for the following
reason:

> The reason behind this was to be able to use the shortcode to favour
> the readability on certain terminals that do not support Emojis!
> @carloscuesta

Terminals are considerably more advanced on all platforms and the
standard Gitmoji set is simple enough that all terminals would display
them correctly nowadays.

This change updates the default format from `code` to `emoji`.

The benefits of this change:

- Improve display of emojis in commit titles, pull request titles and
  list of pull requests on some remote repository providers.

- Removes need to modify the pager to allow `git log` to display
  emojis.

Issue: #1247
Signed-off-by: Michael Lorant <michael.lorant@nine.com.au>
@mikelorant mikelorant mentioned this pull request Jan 28, 2024
3 tasks
@mikelorant
Copy link
Contributor Author

@carloscuesta Can you approve this workflow? Thanks.

Copy link
Owner

@carloscuesta carloscuesta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Thanks 👏🏼

Copy link

codecov bot commented Jan 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f97be59) 91.75% compared to head (0cb2deb) 91.75%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1248   +/-   ##
=======================================
  Coverage   91.75%   91.75%           
=======================================
  Files          28       28           
  Lines         279      279           
  Branches       66       66           
=======================================
  Hits          256      256           
  Misses         22       22           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@carloscuesta carloscuesta merged commit 58961be into carloscuesta:master Jan 28, 2024
3 checks passed
@mikelorant mikelorant deleted the feat/emoji-commit-format branch January 28, 2024 21:44
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review default emoji format
2 participants