search

tc39 / proposal-intl-duration-format

https://tc39.es/proposal-intl-duration-format
MIT License
163 stars 18 forks source link

restore Resolved Options table to ordering before PR #150 merged #165

Closed ben-allen closed 1 year ago

ben-allen commented 1 year ago

fix #163

anba commented 1 year ago

https://github.com/tc39/proposal-intl-duration-format/issues/163#issuecomment-1681674869 has the expected order. That means only [[NumberingSystem]] and [[Locale]] need to be switched.

FrankYFTang commented 1 year ago

#163 (comment) has the expected order. That means only [[NumberingSystem]] and [[Locale]] need to be switched.

This proposal reach Stage 3 in Oct 2021 and at that time the spec is https://github.com/tc39/proposal-intl-duration-format/blob/6e1d1749473d92f7ff5f534e57d93b9a97d51286/spec.emu

the numberingSystem is the last in the table of that version. And from my understanding, there are no consensus after that reach in TC39 to change the order of numberingSystem in resolvedOptions() yet. but somehow in the last merging, that order got accidentially changed and merged.

What I like to see is a "recover" PR to fix that technical mistake during merge.

I am open to discuss what should be the correct order and wiill likely to support a normative PR to put them in the right order as what you suggest, but that should be a different process in term of TC39.