Skip to content

Sticky flag "y", searching at position #424

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.

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

Merged
merged 11 commits into from
Dec 17, 2020

Conversation

AbiF73
Copy link
Contributor

@AbiF73 AbiF73 commented Oct 15, 2020

Hola! Tuve algunas dudas con algunas palabras, lo siento. Espero sus correcciones.

@CLAassistant
Copy link

CLAassistant commented Oct 15, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

bienvenido
y bienvenido si continuas

Sabes de git ? veo que modificaste tu master
No nos afecta en el pr, pero lo mejor para trabajar en tu fork es
dejar el master intocable (solo actualizar desde nuestro fork)
generar en un branch y modificar alli
una vez hecho el pr y añadido al nuestro repo, puedes borrar el branch
y actualizar tu master desde nuestro remoto


We can get all matches in the loop:
Podemos conseguir todas las coincidencias en el loop:

```js run
Copy link
Member

Choose a reason for hiding this comment

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

¿Podrias traducir comentarios lineas 63 64 ?
github no me deja sugerir ahi (dentro de un bloque sin traducir)

Quizas tambien la 62 pero con cuidado:

alert( Se encontró ${result[0]} en la posición ${result.index} );

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@joaquinelio joaquinelio changed the title Update article.md Oct 15, 2020
@joaquinelio
Copy link
Member

ah, para llevar el control (y Don Bot lo entienda)
cambié el nombre al PR

Sticky flag "y", searching at position

@joaquinelio
Copy link
Member

AAAHHHHHHH!!!!

Ilya marcó la version en ingles como WIP
el crazy va a rescribirlo todo!!!!

@vplentinax
Copy link
Contributor

Después que se acepten las sugerencias de Joaquinelio, paso yo a revisar.

@vplentinax vplentinax self-requested a review October 25, 2020 16:08
@joaquinelio
Copy link
Member

Después que se acepten las sugerencias de Joaquinelio, paso yo a revisar.

@vplentinax
El pr es de hace 10 dias, las suggest son de hace 10 dias...

Encima Ilya lo rescribió dos dias después.
Aun no se si hacer sync (ingles-ingles, seria directo) y que este quede con conflictos multiples
o esperar al merge de esto e descartar la nueva version para verla tranquilo

@vplentinax
Copy link
Contributor

@joaquinelio Hazlo como te parezca más cómodo a ti. No tengo inconveniente con ninguna de las dos formas.

@joaquinelio
Copy link
Member

@vplentinax
Este es un art breve
El ingles fue reescrito o rescrito, (en repo esp deje version vieja para que esto no diera conflictos)
podriamos descartar esto y hacerlo de cero

por otro lado,
corrigiendo ortografia le daria MERGE,
es un art muy facil de entender
e Ilya hizo mucho cambio no dire superfluo pero...

vplentinax and others added 9 commits December 17, 2020 13:14
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
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.

Ok

@joaquinelio joaquinelio merged commit 5944323 into javascript-tutorial:master Dec 17, 2020
@javascript-translate-bot

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 participants