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.

Sign up for GitHub

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

Jump to bottom

Arrays #309

Merged
merged 28 commits into from Jul 22, 2020
Merged

Arrays #309

merged 28 commits into from Jul 22, 2020

Conversation

joaquinelio
Copy link
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
Copy link
Contributor

Si, corchetes!

maksumi
maksumi approved these changes Jul 21, 2020
View reviewed changes
Copy link
Collaborator

@maksumi maksumi left a comment

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...

1-js/05-data-types/04-array/10-maximal-subarray/solution.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/10-maximal-subarray/solution.md Show resolved Hide resolved
1-js/05-data-types/04-array/10-maximal-subarray/solution.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/10-maximal-subarray/solution.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/10-maximal-subarray/solution.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/04-array/article.md Outdated Show resolved Hide resolved
@vplentinax
Copy link
Contributor

vplentinax commented Jul 21, 2020 •
edited

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
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
5314f8f 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
b3a0cc0 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md
af2e2a0 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
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
Update 1-js/05-data-types/04-array/article.md
baa16d7 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
71793c9 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
1a517fd 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
a73a204 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
e88e22a 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/5-array-input-sum/task.md
fef7302 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/2-create-array/task.md
93aedf5 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/3-call-array-this/solution.md
cc3750c 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/3-call-array-this/task.md
c2f4f67 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
05a0618 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/5-array-input-sum/solution.md
383249d 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
f122d1e 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
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
Update 1-js/05-data-types/04-array/article.md
726df56 
AAAAHHHHHHH!!!!

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
b3e4585 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
2f951b1 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
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
Update 1-js/05-data-types/04-array/article.md
fe7c1bf 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/05-data-types/04-array/article.md
214685f 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
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
Copy link
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.

@joaquinelio
Update 1-js/05-data-types/04-array/article.md
bcb2dc4 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Copy link
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
Copy link
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
Copy link
Collaborator

@maksumi maksumi left a comment

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
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
Copy link
Member Author

/done

maksumi
maksumi reviewed Jul 22, 2020
View reviewed changes
Copy link
Collaborator

@maksumi maksumi left a comment

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
Copy link
Contributor

@vplentinax vplentinax left a comment

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
Copy link

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.

None yet
4 participants
@joaquinelio @vplentinax @javascript-translate-bot @maksumi