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.

Sign up for GitHub

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

Jump to bottom

ECMA-402 traits discussion #110

Closed
filmil opened this issue Jun 3, 2020 · 4 comments
Closed

ECMA-402 traits discussion #110

filmil opened this issue Jun 3, 2020 · 4 comments
Assignees
@zbraniecki
Labels
A-design Area: Architecture or design C-meta Component: Relating to ICU4X as a whole R-out-of-scope Resolution: ICU4X is the wrong project T-core Type: Required functionality

Comments

@filmil
Copy link
Contributor

filmil commented Jun 3, 2020

This is to point anyone interested in the direction of this pull request: google/rust_icu#110

It's where @zbraniecki is helping me prototype an ECMA-402 inspired abstract API, based on my earlier proposal.

Feel free to chime in if you want. It's been a fun journey so far, most recommended.
@sffc sffc added A-design Area: Architecture or design C-meta Component: Relating to ICU4X as a whole T-core Type: Required functionality labels Jun 4, 2020
@filmil
Copy link
Contributor Author

filmil commented Jun 12, 2020

FYI google/rust_icu#125 contains a Rust implementation for Intl.ListFormat, backed by rust_icu.

@sffc
Copy link
Member

sffc commented Jun 17, 2020

Is this issue actionable within the ICU4X repo? If not, I'd like to close it.

@filmil
Copy link
Contributor Author

filmil commented Jun 17, 2020

Since for now the ECMA-402 implementation work doesn't touch icu4x components, if you want to declutter the issues dashboard, it is OK to close this one.

Be aware though, that implementing ECMA-402 traits will probably require us to make some API adjustment on icu4x so that it can implement the traits. At that point I think it will be appropriate to open a new issue for that work.

@filmil filmil closed this as completed Jun 17, 2020
@zbraniecki
Copy link
Member

At that point I think it will be appropriate to open a new issue for that work.

Agree.

@sffc sffc added the R-out-of-scope Resolution: ICU4X is the wrong project label Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zbraniecki zbraniecki

Labels
A-design Area: Architecture or design C-meta Component: Relating to ICU4X as a whole R-out-of-scope Resolution: ICU4X is the wrong project T-core Type: Required functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@filmil @zbraniecki @sffc