Skip to content

Remove emoji from header#47

Merged
mpkorstanje merged 1 commit intomainfrom
mpkorstanje-patch-2
Aug 9, 2024
Merged

Remove emoji from header#47
mpkorstanje merged 1 commit intomainfrom
mpkorstanje-patch-2

Conversation

@mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Aug 8, 2024

Github automatically converts headers in Markdown to HTML links.

Unfortunately this includes any emoji included in the header. So

### Get in touch πŸ™‹β€β™€οΈ

is linked to as https://github.com/cucumber#get-in-touch-%EF%B8%8F which is hard to parse and easy to break should the the emoji change.

Removing the emoji will make cucumber/common#2195 a bit easier, but also remove some color.

Github automatically converts headers in Markdown to HTML links.

Unfortunately this includes any emoji included in the header. So

is linked to as `https://github.com/cucumber#get-in-touch-%EF%B8%8F` which is hard to parse and easy to break should the the emoji change.
@mpkorstanje mpkorstanje force-pushed the mpkorstanje-patch-2 branch from a8afb06 to ca70963 Compare August 9, 2024 13:45
@mpkorstanje mpkorstanje changed the title Remove emoji from headers Remove emoji from header Aug 9, 2024
@mpkorstanje mpkorstanje merged commit fc6dfc9 into main Aug 9, 2024
@mpkorstanje mpkorstanje deleted the mpkorstanje-patch-2 branch August 9, 2024 13:46
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

Successfully merging this pull request may close these issues.

1 participant

Comments