Skip to content

Conversation

@boykovdmitriy
Copy link

Hello. I needed to use this library and I rewrote it for my needs. Hope you'll like it)

  • Rewrote to ES6 classes
  • Added separate imports for languages from 'written-number/i18n'
  • Refactored code a little)
  • Changed code coverage tool to Istanbul
  • Changed building tool from 'Grunt' to webpack
  • Added eslint with airbnb rules
  • Changed readme

export {
Ar,
Az,
En,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep language names in lower case for preparation for standardisation. The target is to use ISO 639-1 code for language names, in combination with country code. It would look like en_US, en_UK, fr, fr_FR.
cf. Locales and Languages Supported by Facebook and num2words

"eslint": "^6.0.1",
"eslint-config-airbnb-base": "^13.2.0",
"eslint-plugin-import": "^2.18.2",
"mocha": "^3.5.3",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we are dropping support node versions less than 8, can you please verify

  • we are using latest versions for devDependencies
  • remove unused dependencies from package.json (may be mocha ?)

@forzagreen
Copy link
Collaborator

Hi @boykovdmitriy ,
I started new library: https://github.com/forzagreen/n2words that's doing the same thing.
It's supporting 18 languages and still growing.
Please check, and feel free to contribute.

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.

2 participants