Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] Small adjustments for pt_BR #110

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

aricaldeira
Copy link

Line #45 - Using comma to separate groups:
spellout -l pt-BR 123456789
before: cento e vinte e três milhões quatrocentos e cinquenta e seis mil setecentos e oitenta e nove
after: cento e vinte e três milhões, quatrocentos e cinquenta e seis mil, setecentos e oitenta e nove

Line #141 - Adding ordinal zero
spellout -l pt-BR -p ordinal 0
before:
after: zerésimo

Deleted line #187 - Portuguese doesn't use "primer" like Spanish

Lines #187 and #189 - Brazil doesn't add the dot before de ª and º symbols
spellout -l pt-BR -p ordinal-number 10
before: 10.º
after: 10º

aricaldeira and others added 2 commits February 3, 2023 10:06
Line Numbertext#45 - Using comma to separate groups:
    spellout -l pt-BR 123456789
    before: cento e vinte e três milhões quatrocentos e cinquenta e seis mil setecentos e oitenta e nove
    after: cento e vinte e três milhões, quatrocentos e cinquenta e seis mil, setecentos e oitenta e nove

Line #141 - Adding ordinal zero
    spellout -l pt-BR -p ordinal 0
    before: 
    after: zerésimo

Deleted line #187 - Portuguese doesn't use "primer" like Spanish

Lines #187 and #189 - Brazil doesn't add the dot before de ª and º symbols
    spellout -l pt-BR -p ordinal-number 10
    before: 10.º
    after: 10º
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant