Skip to content

allow number >= 20 for hundred#112

Closed
nmorel wants to merge 1 commit intoallo-media:masterfrom
nmorel:fix/hundred-multiplier
Closed

allow number >= 20 for hundred#112
nmorel wants to merge 1 commit intoallo-media:masterfrom
nmorel:fix/hundred-multiplier

Conversation

@nmorel
Copy link

@nmorel nmorel commented Apr 2, 2024

Fixes #111

@nmorel
Copy link
Author

nmorel commented Apr 2, 2024

One regression in french with this more "laxist" change, visible in the unit test.
I can make an english parser if needed.

@rtxm rtxm self-assigned this Apr 2, 2024
@rtxm rtxm added the enhancement New feature or request label Apr 2, 2024
@rtxm
Copy link
Collaborator

rtxm commented Apr 16, 2024

I propose an alternative solution in #113 that doesn't impact other languages and doesn't need a specific parser.

@nmorel
Copy link
Author

nmorel commented Apr 16, 2024

@rtxm That's great, thanks !
I'm closing this one then.

@nmorel nmorel closed this Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

2 participants