Skip to content
This repository has been archived by the owner on May 12, 2020. It is now read-only.

Update to Unicode 11.0 [breaking change]. #2

Closed
wants to merge 4 commits into from

Conversation

mbrubeck
Copy link
Contributor

This is a breaking change because the variants of the public Script enum have changed. Code that matches this enum exhaustively must be updated.

Depends on servo/rust-harfbuzz#78.

@bors-servo
Copy link

☔ The latest upstream changes (presumably #3) made this pull request unmergeable. Please resolve the merge conflicts.

This is a breaking change because the variants of the public `Script`
enum have changed.  Code that matches this enum exhaustively must be
updated.
@mbrubeck
Copy link
Contributor Author

r? @pcwalton

@behnam
Copy link

behnam commented May 18, 2018

Script property is a Unicode Catalog type and expected to change on every single Unicode release.

How about we mark the Enum as open now, so we don't have to call it breaking because of this again?

@apendleton
Copy link

11.0 is out now, so may as well skip straight to that? http://unicode.org/versions/Unicode11.0.0/

@pcwalton
Copy link

pcwalton commented Jun 8, 2018

Agreed with @behnam.

@mbrubeck
Copy link
Contributor Author

Updated to Unicode 11.0 and made the enum open.

Note: The harfbuzz-sys crate hasn't yet been updated to a version of Harfbuzz that includes Unicode 11.0. Once it is, this can be updated to support the new HB_SCRIPT constants.

@mbrubeck mbrubeck changed the title Update to Unicode 10.0 [breaking change]. Update to Unicode 11.0 [breaking change]. Jun 19, 2018
@bors-servo
Copy link

☔ The latest upstream changes (presumably #4) made this pull request unmergeable. Please resolve the merge conflicts.

@RazrFalcon
Copy link

Version 12 available.

@mbrubeck mbrubeck closed this Jan 2, 2020
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.

6 participants