Plural: Support ordinals

[tniessen]

Fixes #403

[rxaviers]

Please, could you move options above value? Also, replace >> with > only.

[rxaviers]

Hi @tniessen, I left a couple of comments above. Great job so far.

[rxaviers]

Let's create a similar table for ordinals?

[rxaviers]

👍

[jzaefferer]

I'd like to see a brief definition for both cardinals and ordinals somewhere in the documentation. More examples would also help.

[tniessen]

@rxaviers @jzaefferer I pushed a commit a few days ago, awaiting reviews.

[rxaviers]

All table needs the below change.

- plural( 0 )
+ plural( 0, { type: "ordinal" } )

[rxaviers]

Hi @tniessen, I added a couple of comments. Github doesn't send notifications when new commits are pushed. So, feel free to add a new comment if you have further updates. Thanks so far.

[tniessen]

@rxaviers Agree with both, new code is almost identical with your suggestion. Updated as 583ae45.

[rxaviers]

Thank you @tniessen, one last comment.

[tniessen]

@rxaviers Fixed in 7538dec, thank you for reviewing!

[rxaviers]

spaces MakePlural.rules[ type ] =

[tniessen]

@rxaviers My bad, sorry... c47456f

[rxaviers]

@tniessen, no problem. Thanks landed 🎉