google / google/rust_icu

Require Debug+Display, not Error

Open
#207 4 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
140
Forks
34
Avg merge
5h 5m
Merged PRs (30d)
4

Description

In ICU4X we're moving towards `no_std` support, and this means that _by default_ the `Error` trait is not implemented. It's possible to explicitly enable the `"std"` feature and have it be enabled, however it seems like it's unnecessary to require `Error` on the error associated types since none of `Error`s functionality is actually used — perhaps we should just require `Debug+Display`, or maybe even just one or none of those traits. In general we should require the traits we need.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.