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

Update Emoji dataset to Unicode 14 #22342

Merged
merged 1 commit into from
Jan 4, 2023
Merged

Conversation

mrsdizzie
Copy link
Member

Gitea emoji dataset was out of date because it gets manually built and hasn't been rebuilt since it was added. This means Gitea doesn't recognize some newer emoji or changes to existing ones.

After changing the max unicode version to 14 I just ran: go run build/generate-emoji.go

This should address the initial issue seen in #22153 where Gitea doesn't recognize a standard alias used elsewhere when importing content.

14 is the latest supported version from the upstream source as 15 is not widely supported (in their opinion) yet: github/gemoji#244 (comment)

Gitea emoji dataset was out of date because it gets manually build and hasn't been rebuilt since it was added. This means Gitea doesn't recognize some newer emoji or changes to existing ones.

After changing the max unicode version to 14 I just ran: `go run build/generate-emoji.go`

This should address the initial issue seen in go-gitea#22153 where Gitea doesn't recognize a standard alias used elsewhere when importing content.

14 is the latest supported version from the upstream source as 15 is not widely supported (in their opinion) yet: github/gemoji#244 (comment)
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 4, 2023
@jolheiser jolheiser added this to the 1.19.0 milestone Jan 4, 2023
@jolheiser
Copy link
Member

fwiw I'm fine with this being backported to 1.18 as a simple dep change.

@jolheiser jolheiser added the outdated/backport/v1.18 This PR should be backported to Gitea 1.18 label Jan 4, 2023
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 4, 2023
@jolheiser jolheiser merged commit f41ad34 into go-gitea:main Jan 4, 2023
@jolheiser
Copy link
Member

Feel free to backport, or I can if you would prefer! 🤝

mrsdizzie added a commit to mrsdizzie/gitea that referenced this pull request Jan 4, 2023
Gitea emoji dataset was out of date because it gets manually built and
hasn't been rebuilt since it was added. This means Gitea doesn't
recognize some newer emoji or changes to existing ones.

After changing the max unicode version to 14 I just ran: `go run
build/generate-emoji.go`

This should address the initial issue seen in go-gitea#22153 where Gitea doesn't
recognize a standard alias used elsewhere when importing content.

14 is the latest supported version from the upstream source as 15 is not
widely supported (in their opinion) yet
@mrsdizzie
Copy link
Member Author

Feel free to backport, or I can if you would prefer! 🤝

Thx! I Created backport (had to remember how 😅

jolheiser pushed a commit that referenced this pull request Jan 4, 2023
Backport of #22342 to release/v1.18 as requested
@jolheiser jolheiser added the backport/done All backports for this PR have been created label Jan 4, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jan 5, 2023
* upstream/main:
  Rewrite fuzzers to native Go harnesses (go-gitea#22313)
  Update Emoji dataset to Unicode 14 (go-gitea#22342)
  fix gravatar disable bug (go-gitea#22336)
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. outdated/backport/v1.18 This PR should be backported to Gitea 1.18
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants