Skip to content

Fix ICU4X data#478

Merged
taj-p merged 3 commits intolinebender:mainfrom
robertbastian:data
Dec 9, 2025
Merged

Fix ICU4X data#478
taj-p merged 3 commits intolinebender:mainfrom
robertbastian:data

Conversation

@robertbastian
Copy link
Contributor

@robertbastian robertbastian commented Dec 9, 2025

  • Clean up old deps
  • Use .with_segmenter_models to remove segmentation models, instead of setting the language to en
  • Set ICU4X dependencies to ~ as baked data is only compatible within minor versions

taj-p
taj-p approved these changes Dec 9, 2025
View reviewed changes
Copy link
Contributor

@taj-p taj-p left a comment

Choose a reason for hiding this comment

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

Thank you 🙏

@taj-p taj-p added this pull request to the merge queue Dec 9, 2025
Merged via the queue into linebender:main with commit 51ffe5d Dec 9, 2025
24 checks passed
@robertbastian robertbastian deleted the data branch December 9, 2025 19:13
@xStrom xStrom mentioned this pull request Jan 11, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@taj-p taj-p taj-p approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@robertbastian @taj-p

Comments