Skip to content

Move unicodeset_parse to icu_properties#7935

Merged
robertbastian merged 4 commits intounicode-org:mainfrom
robertbastian:unicode-set
May 5, 2026
Merged

Move unicodeset_parse to icu_properties#7935
robertbastian merged 4 commits intounicode-org:mainfrom
robertbastian:unicode-set

Conversation

@robertbastian
Copy link
Copy Markdown
Member

@robertbastian robertbastian commented May 4, 2026

#3959

Changelog

icu_properties:

  • Add experimental UTS#35 Unicode set parsing

@robertbastian robertbastian requested a review from a team as a code owner May 4, 2026 15:26
mod harfbuzz;

#[cfg(feature = "unstable")]
pub mod unicodeset_parse;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: should be called unstable_

also needs scary unstable header

@robertbastian robertbastian requested a review from sffc as a code owner May 4, 2026 17:57
@robertbastian robertbastian requested a review from Manishearth May 4, 2026 18:01
Copy link
Copy Markdown
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

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

Seems fine. This doesn't seem to be modifying provider/source, but when that happens we will need to access this code through doc(hidden) pub mod unstable_internal so that datagen doesn't need unstable features.

@robertbastian
Copy link
Copy Markdown
Member Author

so that datagen doesn't need unstable features

datagen already uses unstable features.

@robertbastian robertbastian merged commit 4570575 into unicode-org:main May 5, 2026
34 of 36 checks passed
@robertbastian robertbastian deleted the unicode-set branch May 5, 2026 09:14
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.

2 participants