Skip to content

Arrays - #309

Merged
joaquinelio merged 28 commits into
javascript-tutorial:masterfrom
joaquinelio:arr
Jul 22, 2020
Merged

Arrays#309
joaquinelio merged 28 commits into
javascript-tutorial:masterfrom
joaquinelio:arr

Conversation

@joaquinelio

@joaquinelio joaquinelio commented Jul 19, 2020

Copy link
Copy Markdown
Member

Usé "corchetes" para "square brackets", como aprendí en el colegio ¿es lo usual?

@javascript-translate-bot
javascript-translate-bot requested a review from a team July 19, 2020 22:55
@iliakan iliakan mentioned this pull request Jul 19, 2020
Open
@vplentinax

vplentinax commented Jul 20, 2020

Copy link
Copy Markdown
Contributor

Si, corchetes!

maksumi
maksumi approved these changes Jul 21, 2020
View reviewed changes

@maksumi maksumi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Algunos typos y algunas sugerencias conceptuales. Na'más...

Comment thread 1-js/05-data-types/04-array/10-maximal-subarray/solution.md Resolved
Comment thread 1-js/05-data-types/04-array/10-maximal-subarray/solution.md Resolved
Comment thread 1-js/05-data-types/04-array/10-maximal-subarray/solution.md Resolved
Comment thread 1-js/05-data-types/04-array/10-maximal-subarray/solution.md Resolved
Comment thread 1-js/05-data-types/04-array/10-maximal-subarray/solution.md Resolved
Comment thread 1-js/05-data-types/04-array/article.md Resolved
Comment thread 1-js/05-data-types/04-array/article.md Resolved
Comment thread 1-js/05-data-types/04-array/article.md Resolved
Comment thread 1-js/05-data-types/04-array/article.md Resolved
Comment thread 1-js/05-data-types/04-array/article.md Resolved
@vplentinax

vplentinax commented Jul 21, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

Algunos typos y algunas sugerencias conceptuales. Na'más...

@maksumi todo lo que puedas corregir o sugerir, incluido typos, añadelos o comenta la línea para comparar.

Edición:

Lo vi desde el cell y no me di cuenta que habías hecho las sugerencias! Sorry!

joaquinelio and others added 21 commits July 21, 2020 06:09
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
5314f8f 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
b3a0cc0 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
af2e2a0 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/5-array-input-sum/task.md
a373976 
Maldito teclado.  La s me está fallando.

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
baa16d7 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
71793c9 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
1a517fd 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
a73a204 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
e88e22a 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/5-array-input-sum/task.md
fef7302 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/2-create-array/task.md
93aedf5 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/3-call-array-this/solution.md
cc3750c 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/3-call-array-this/task.md
c2f4f67 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
05a0618 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/5-array-input-sum/solution.md
383249d 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
f122d1e 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
8fdb93a 
ocioso...
pero me da igual

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
726df56 
AAAAHHHHHHH!!!!

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
b3e4585 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
2f951b1 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
785ff52 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
joaquinelio and others added 4 commits July 21, 2020 08:59
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
fe7c1bf 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
214685f 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
db4e7fb 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update article.md
778666a
@joaquinelio

joaquinelio commented Jul 21, 2020

Copy link
Copy Markdown
Member Author

/done
pero no del todo, un par de suggest aun por resolver @maksumi @vplentinax

De acuerdo, longitud y largo es lo mismo pero longitud se lee más claro y cómodo.
si hubiera sabido que eran tantos habria editado con /reemplazar.
LISTO lo hice, encontre 2 más. irrelevantess, pero para concordar.

ingreasar por entrar, ok, se me escapo

¿Te molesta la voz pasiva?
La evito si está de más pero a veces ayuda en la secuencia de lo que se expresa.
Aca me daba igual.

Te desprecié dos o tres, alguno para debate.
habrás de leer...
1 5 4 10 solution 40, 66

1 5 4 article 124 136 190 editadas sin el suggest,
lo que muestra la conversación no es lo que quedó.

¿Deberia poner mayusc y punto final en los // ? nah. Pero me da nosequé.

monton de commits, debi usar modo batch.

maksumi
maksumi reviewed Jul 21, 2020
View reviewed changes
Comment thread 1-js/05-data-types/04-array/article.md Resolved
@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/article.md
bcb2dc4 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio

joaquinelio commented Jul 22, 2020

Copy link
Copy Markdown
Member Author

@vplentinax ,pido tu opinión españolísima (aunque seas implantada)
1 5 4 10 solution, 2 ssuggest que desprecié.
Uno es la coma, no me importa quitarla pero creo que va.
Segundo, el uso de "actual" para ssignificar lo que esttá en curso en lugar del que creí más ajustado "corriente", ( y ya está explicando código, por eso me detengo a pensarlo)
pero noté que en otro lado también usaron "actual"... Casi preferiría "elemento presente".
¿te parece que lo cambie a "actual"?

@vplentinax

vplentinax commented Jul 22, 2020

Copy link
Copy Markdown
Contributor

@vplentinax ,pido tu opinión españolísima (aunque seas implantada)
1 5 4 10 solution, 2 ssuggest que desprecié.
Uno es la coma, no me importa quitarla pero creo que va.
Segundo, el uso de "actual" para ssignificar lo que esttá en curso en lugar del que creí más ajustado "corriente", ( y ya está explicando código, por eso me detengo a pensarlo)
pero noté que en otro lado también usaron "actual"... Casi preferiría "elemento presente".
¿te parece que lo cambie a "actual"?

Hecho!

maksumi
maksumi approved these changes Jul 22, 2020
View reviewed changes

@maksumi maksumi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ya aprové... Lo dejo a tu criterio porque la traducción está muy bien y solo son sugerencias alternativas que a mi criterio creo que funcionarían también

@joaquinelio Maksumi Murakami
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
6a6b3f8 
NO creo que sea mejor "actual".  Pero busqué en el repo, y fue lo usado.  Sea por correcto o por convención, pensando en quien aprende, prefiero la homogeneidad.  Va ACTUAL.

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio

joaquinelio commented Jul 22, 2020

Copy link
Copy Markdown
Member Author

/done

maksumi
maksumi reviewed Jul 22, 2020
View reviewed changes

@maksumi maksumi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Okay, creo que solo nos falta la aprobación de @vplentinax y merge!

vplentinax
vplentinax approved these changes Jul 22, 2020
View reviewed changes

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes!

@joaquinelio
joaquinelio merged commit ef97645 into javascript-tutorial:master Jul 22, 2020
@javascript-translate-bot

javascript-translate-bot commented Jul 22, 2020

Copy link
Copy Markdown

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

@joaquinelio
joaquinelio deleted the arr branch July 22, 2020 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maksumi maksumi maksumi approved these changes
@vplentinax vplentinax vplentinax approved these changes

Assignees

No one assigned

Labels

ready to merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@joaquinelio @vplentinax @javascript-translate-bot @maksumi