Skip to content

Comments

import: Use mattn/go-runewidth instead of zyedidia/go-runewidth#3601

Merged
JoeKar merged 2 commits intomicro-editor:masterfrom
JoeKar:import/go-runewidth
Jan 6, 2025
Merged

import: Use mattn/go-runewidth instead of zyedidia/go-runewidth#3601
JoeKar merged 2 commits intomicro-editor:masterfrom
JoeKar:import/go-runewidth

Conversation

@JoeKar
Copy link
Member

@JoeKar JoeKar commented Jan 5, 2025

The upstream version includes the changes added on top after the fork of zyedidia/go-runewidth.

#3601 (comment):

[...] both our downstream changes zyedidia/go-runewidth#1 and zyedidia/go-runewidth#2 were merged upstream in mattn/go-runewidth#44 and mattn/go-runewidth#47.

Fixes #3264

Copy link
Contributor

@Andriamanitra Andriamanitra left a comment

Choose a reason for hiding this comment

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

This also fixes #3264 as the upstream version now includes Unicode 15 support

@dmaluka
Copy link
Collaborator

dmaluka commented Jan 5, 2025

It would be worth mentioning in the PR description that both our downstream changes zyedidia/go-runewidth#1 and zyedidia/go-runewidth#2 were merged upstream in mattn/go-runewidth#44 and mattn/go-runewidth#47.

@JoeKar JoeKar merged commit c77ed02 into micro-editor:master Jan 6, 2025
6 checks passed
@JoeKar JoeKar deleted the import/go-runewidth branch January 6, 2025 06:07
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.

New emojis (2021 Unicode 14.0+) cause rendering issues

3 participants