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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: properly render 馃毊 in default templates (cherry-pick #4044) #4049

Merged
merged 1 commit into from Dec 20, 2023

Conversation

gcp-cherry-pick-bot[bot]
Copy link

Cherry-picked Fix rendering of 馃毊 in default templates (#4044)

It seems that GitHub has changed its Markdown renderer such that an
emoji followed by a string of plain text (with no other formatting)
followed by a colon at the end of the line renders as plain text instead
of as the emoji. Compare the following two lines:

  • 馃毊 To delete all plans and locks for the PR, comment:

  • 馃毊 To delete all plans and locks for the PR, comment:

While this seems like it is probably a GitHub bug, we can work around it
by adding some formatting between the emoji and the end of the line,
which matches the bullet point above it anyway.

@gcp-cherry-pick-bot gcp-cherry-pick-bot bot requested a review from a team as a code owner December 13, 2023 14:56
@GenPage GenPage added bug Something isn't working templates labels Dec 13, 2023
@GenPage GenPage added this to the v0.27.1 milestone Dec 13, 2023
It seems that GitHub has changed its Markdown renderer such that an
emoji followed by a string of plain text (with no other formatting)
followed by a colon at the end of the line renders as plain text instead
of as the emoji. Compare the following two lines:

* 馃毊 To delete all plans and locks for the PR, comment:

* 馃毊 To **delete** all plans and locks for the PR, comment:

While this seems like it is probably a GitHub bug, we can work around it
by adding some formatting between the emoji and the end of the line,
which matches the bullet point above it anyway.
@lukemassa lukemassa force-pushed the cherry-pick-0c5d97-release-0.27 branch from ba0a2cd to 6535bf0 Compare December 20, 2023 16:43
@lukemassa lukemassa merged commit 6ebe2b8 into release-0.27 Dec 20, 2023
22 of 23 checks passed
@lukemassa lukemassa deleted the cherry-pick-0c5d97-release-0.27 branch December 20, 2023 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants