i18n

[alanmtk]

Implemented node-i18n and added es/en locales

[josx]

Varias cosas para remarcar:

• Primero que nada esta muy bien usar la lib i18n que es muy simple y parece ser bastante liviana

• El segundo commit no debería existir ya que es algo que deberia estar bien hecho en primer momento, ahi deberias hacer un squash de ese commit.

• El repo tiene el build de la aplicación commiteado, obviamente no es una buena práctica. Deberiamos igualmente pensar que hacer con esto (si preguntar al programador, mandarle el build o como bien hiciste no ponerlo)

• No me funciona:

josx@idilia /tmp/node-games⚡git:i18n* $ DEBUG=i18n:* node index.js
  i18n:debug will use /tmp/node-games/locales/en.json +0ms
  i18n:debug read /tmp/node-games/locales/en.json for locale: en +3ms
  i18n:debug will use /tmp/node-games/locales/es.json +0ms
  i18n:debug read /tmp/node-games/locales/es.json for locale: es +0ms
usage: node-games <game>

  i18n:warn WARN: No locale found - check the context of the call to __(). Using en as current locale +2ms
Games
- spacecraft
- snake
- tanks
josx@idilia /tmp/node-games⚡git:i18n* $ locale
LANG=es_AR.UTF-8
LANGUAGE=es_AR:es
LC_CTYPE="es_AR.UTF-8"
LC_NUMERIC="es_AR.UTF-8"
LC_TIME="es_AR.UTF-8"
LC_COLLATE="es_AR.UTF-8"
LC_MONETARY="es_AR.UTF-8"
LC_MESSAGES="es_AR.UTF-8"
LC_PAPER="es_AR.UTF-8"
LC_NAME="es_AR.UTF-8"
LC_ADDRESS="es_AR.UTF-8"
LC_TELEPHONE="es_AR.UTF-8"
LC_MEASUREMENT="es_AR.UTF-8"
LC_IDENTIFICATION="es_AR.UTF-8"
LC_ALL=

[josx]

Si en todos lados estan usando single quote deberias usar single quote, no double quote.

[alanmtk]

Opté por remover ese import, ya no era necesario porque es importado en index.js

[josx]

genial

[josx]

¿Esto funciona en windows? o habría que por las dudas usar
path.join o similar.

[alanmtk]

Lo cambié por por path.join(__dirname, '/locales') que nos saca el problema de los slashes según sistema operativo

[josx]

perfecto

[alanmtk]

@josx No te esta funcionando porque no están las cosas buildeadas como originalmente vienen en el repo, en un principio no lo subí por una cuestión de que no es buena practica pero si es cierto que sin eso y siguiendo las instrucciones del README no funcionaría si lo descargan. En cuanto a lo de los multiples commits desde la pull puedo hacer un "Squash and merge" que junta todos los commits y mergea, es esa la idea o debería encararlo de otro modo?

[josx]

• Todos los que son Arreglos ej: fix deberias usar squash o fixup, y tener separados los que creas que son claramente algo en si mismo pertinente de marcar. (luego podemos charlar sobre las diferencias y como se hace esto úlitmo sin github con in git rebase interactivo)

• Sobre el funcionamiento ¿Como lo estas probando vos?