Methods of RegExp and String

[dbritto-dev]

No description provided.

[joaquinelio]

Mmm....
No es "draft"
Pero se ve un poco "messed up"

"Methods of RegExp and String" sería una buena adición, ¿esta lissta para rev?
pero tiene partes incompletas de otros articulos
Es porque no hiciste un nuevo branch para cada articulo,
el pr trata de meter TODOS los cambios que teness pendientes

que trabajes sobre tu master no nos afecta si haces uno a la vez y luego lo actualizas antes de empezar otro,
Pero la manera correcta de trabajar con git es:

-Nunca tocar tu master, solo actualizarlo con pull (o fetch/merge)
-Para modificar 1 articulo (o funcionalidad en el codigo), crear branch, hacer todas las modificaciones alli, generar pr.
-Para modificar OTRO articulo (u otra parte de codigo) crear OTRO branch, modificar y hacer OTRO pr.

Jaja, me paso lo mismo hace un año... #105
en el repo tenes las explicaciones de como crear fork y branch
¿trabajas online o local en un pc tuyo?

Si sos curioso:
Si miras #105 alguien me ayudo con una solución GIT (git abarca todo, controla todo, podes deshacer todo, rehacer todo, corregir todo, y registra todo)
Pero decidi copiar los dos textos, empezar sobre el fork limpio, generar dos ramas y pegar cada texto en la rama respectiva, y luego hice un pr de cada rama. Git reconocio el texto pegado como modificacion, no como archivo nuevo porque no le importa el origen, compara linea a linea y registra los cambios en un commit.

O TAMBIEN
Y por esta unica vez
podrias terminar "styles and classes" y hacemos revisiones al mismo tiempo (y lo agregamos a mano al issue17), no me molestaria para nada (solo no agregues nuevos articulos hasta arreglar esto)

no sé que piensa/pensará el eventual 2do reviewer...
hello...

[joaquinelio]

Ah, tampoco hace falta tener PR "wip" o draft (salvo que estes invitando a otros a modificarlo?) , los cambios parciales estan seguros dentro de tu branch ( que en este caso es master).

[dbritto-dev]

@joaquinelio puede hacer una copia del origen y hacer cherry pick a cada commit cerrar este y crear otro PR

[joaquinelio]

Revisando el PR,
Hay poco hecho de "styles"

tambien podrias deshacer eso (podes copiar la tarea que tradujiste) y hacer un nuevo pr con "regexp str" solamente
esperas la revision/merge y a partir de alli limpias todo y empiezas a trabajar con branches como experto, con git añadido a tu Curriulum Vitae. =)

[joaquinelio]

ah no te habia leido
a ver...

[dbritto-dev]

ah no te habia leido
a ver...

lo de Styles and classes esta en draft

[dbritto-dev]

Revisando el PR,
Hay poco hecho de "styles"

tambien podrias deshacer eso (podes copiar la tarea que tradujiste) y hacer un nuevo pr con "regexp str" solamente
esperas la revision/merge y a partir de alli limpias todo y empiezas a trabajar con branches como experto, con git añadido a tu Curriulum Vitae. =)

@joaquinelio descuida este se envió mal por alguna razón, recien veo que la rama dice master jajaja. Por otro lado no entendí lo de experto y lo de git añadido a mi CV, no hay que ser experto para eso.

[joaquinelio]

"styles" en draft, en otro pr, ok, para que no perdamos tiempo, gracias.

pero como trabajaste sobre tu master, estan reflejados aca tambien "files 3"
Si hiciera MERGE se agregarian esos incompletos

No tengo problemas que trabajes sobre tu master mientras no implique trabajo extra como bajar tu rama y hacer cherry pick y otro pr,
la manera de menor trabajo para todos es trabajar sobre branch

[dbritto-dev]

"styles" en draft, en otro pr, ok, para que no perdamos tiempo, gracias.

pero como trabajaste sobre tu master, estan reflejados aca tambien "files 3"
Si hiciera MERGE se agregarian esos incompletos

No tengo problemas que trabajes sobre tu master mientras no implique trabajo extra como bajar tu rama y hacer cherry pick y otro pr,
la manera de menor trabajo para todos es trabajar sobre branch

Lo del cherry pick es sencillo son unos cuantos commits que hay que copiar y ya.

[joaquinelio]

ah otro coment tuyo que no lei antes de mandar el mio...
ja,
debo ver que esto no tiene formato de chat es mas dificil seguirlo
mmm
me perdi un poco

[joaquinelio]

ah, no te conozco,
asumi que trabajaste sobre master por inexperiencia
me puse en modo tutor jajajj sorry

[joaquinelio]

quizas sepas mas que yo y lo del master y el mix fue solo un error, te entiendo

[joaquinelio]

entonces te pediria que me ayudes...

si ves mi ultimo pr con 58 archivos modificados (que @vplentinax reviso)
entenderas que no quiero siquiera hacer un cherry pick

[dbritto-dev]

entonces te pediria que me ayudes...

si ves mi ultimo pr con 58 archivos modificados (que @vplentinax reviso)
entenderas que no quiero siquiera hacer un cherry pick

jajaja te entiendo, al parece ya se actualizo mi rama master y se fueron los cambios :p tengo que volverlos a recuperar y hacerles copia

[joaquinelio]

si podes hacer algo para ayudar, estoy medio quemado

mandar pr limpio, cerrar este o modificarlo
asi solo nos queda revisar y hacer merge...

y si tenes experiencia js no vendria mal una ayuda a las review...

[joaquinelio]

otra vez...
mande coment sin haber visto el tuyo
break para comer vemos...
como seguimos?

[dbritto-dev]

si podes hacer algo para ayudar, estoy medio quemado

mandar pr limpio, cerrar este o modificarlo
asi solo nos queda revisar y hacer merge...

y si tenes experiencia js no vendria mal una ayuda a las review...

este queda cerrado solo faltaría crear otro PR. Si claro también puedo apoyar a hacer las review.