Skip to content
This repository has been archived by the owner on May 23, 2019. It is now read-only.

A test uses '1' as a plural group. #23

Closed
drewfish opened this issue Dec 17, 2013 · 2 comments
Closed

A test uses '1' as a plural group. #23

drewfish opened this issue Dec 17, 2013 · 2 comments

Comments

@drewfish
Copy link
Contributor

The "complex object formatter with offset" test has a plural match for '1' (tests/index.js line 151). However, the spec says that (a) the format is '=1' and that (b) that use is discouraged.

@apipkin
Copy link
Contributor

apipkin commented Dec 18, 2013

This was in place in the strawman examples http://wiki.ecmascript.org/doku.php?id=globalization:messageformatting#modified_icu_approach

I didn't think this would ever match anywhere. I'll remove this.

@apipkin
Copy link
Contributor

apipkin commented Dec 18, 2013

Fixed in PR #16

@apipkin apipkin closed this as completed Dec 18, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants