icu4x
icu4x copied to clipboard
Consider renaming resolve_components to resolved_components
The documentation says it returns what was resolved, but the name suggests triggering resolution.
This is referring to the experimental DateTimeFormatter API.
A bunch of this will probably change under skeletons anyway.
@zbraniecki to approve.
Also, per discussion elsewhere, the return type should probably be some kind of ResolvedBag that doesn't wrap each field in Option.
Agree. And agree with the last comment, the ResolvedBag will come from icu_preferences.
This API will be changing in 2.0 but I'll make sure to call it resolved_options.
Datetime has a new API for this, which is documented. Might still need to look at other components