Skip to content

Methods of RegExp and String #411

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

Closed
wants to merge 0 commits into from

Conversation

dbritto-dev
Copy link
Contributor

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from a team October 7, 2020 14:22
@dbritto-dev dbritto-dev changed the title WIP: Methods of RegExp and String Methods of RegExp and String Oct 7, 2020
@dbritto-dev dbritto-dev marked this pull request as ready for review October 8, 2020 17:25
@joaquinelio
Copy link
Member

joaquinelio commented Oct 9, 2020

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

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

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

@joaquinelio
Copy link
Member

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

ah no te habia leido
a ver...

@dbritto-dev
Copy link
Contributor Author

ah no te habia leido
a ver...

lo de Styles and classes esta en draft

@dbritto-dev
Copy link
Contributor Author

dbritto-dev commented Oct 9, 2020

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

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

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

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

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

@joaquinelio
Copy link
Member

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

@joaquinelio
Copy link
Member

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

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

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

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

@dbritto-dev
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy