Skip to content
This repository was archived by the owner on Jan 20, 2022. It is now read-only.

weekly-update.2015-03-20.md#120

Merged
stringparser merged 6 commits intonodejs:masterfrom
pandres95:master
May 3, 2015
Merged

weekly-update.2015-03-20.md#120
stringparser merged 6 commits intonodejs:masterfrom
pandres95:master

Conversation

@pandres95
Copy link
Copy Markdown

Traducción del documento weekly-update.2015-03-20.md de acuerdo al #111. Por favor alguien me ayude a revisar si está correctamente traducido.

@stringparser
Copy link
Copy Markdown

Perfect. Hago ping a la gente de revisión.

cc @ipeluffo @edsadr @JoseRoman @italoacasas

@stringparser
Copy link
Copy Markdown

Sip, le pongo otra vez la etiqueta.

@pandres95
Copy link
Copy Markdown
Author

@edsadr, el commit correcto es el 672f3e8

@stringparser
Copy link
Copy Markdown

cierto 💃 :), si la gente de revisión no te lo corrige esta semana hecho yo una mano

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

comandas -> comandos

@stringparser
Copy link
Copy Markdown

El resto esta bien. Excepto el párrafo que falta traducir y los errores que veo está bien traducido para mi 👍.

Otra cosa, veo que has corregido también el CONTRIBUTING.md y sin querer lo has metido en la PR. ¿Puedes deshacer los cambios hechos para que no esté en esta PR? Abre otra si no para eso ;)

@pandres95
Copy link
Copy Markdown
Author

Listo, deshaciendo ese cambio del CONTRIBUTING.md, ¿cómo hago eso? :')

@pandres95
Copy link
Copy Markdown
Author

@stringparser estuvo :)

@stringparser
Copy link
Copy Markdown

haha, di que si @pandres95. Voy a intentar que te lo revise alguien más y si no te haces el merge y luego lo publicas en @medium.

¿Alguien tiene tiempo para echarle un vistazo rápido?
cc @ipeluffo @edsadr @JoseRoman @italoacasas

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Esta se ha suavizado para path.dirname(), path.basename(), y path.extname() (Colin Ihrig) #1216.

La verificación de tipos ha sido relajada para path.dirname(), path.basename(), y path.extname() (Colin Ihrig) #1216.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

s/apropiadamente/propiamente, suena mejor para mi.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • @ripper2hl dijo: optimizciones y se hizo la luz :D

@a0viedo
Copy link
Copy Markdown
Member

a0viedo commented Apr 30, 2015

@pandres95 agregué pequeñas sugerencias, muy buen trabajo 👏

@stringparser
Copy link
Copy Markdown

gracias por la rápida revisión @a0viedo! 😄

@ripper2hl
Copy link
Copy Markdown

optimizciones D:

@stringparser
Copy link
Copy Markdown

hahah, añadido

@pandres95
Copy link
Copy Markdown
Author

@a0viedo @stringparser Listo :)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@pandres95 Falta corregir: optimizciones, lo que señaló @ripper2hl

@stringparser
Copy link
Copy Markdown

great!

stringparser added a commit that referenced this pull request May 3, 2015
weekly-update.2015-03-20.md
@stringparser stringparser merged commit 182f889 into nodejs:master May 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants