- Notifications
You must be signed in to change notification settings - Fork230
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
joaquinelio commentedOct 9, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Mmm.... "Methods of RegExp and String" sería una buena adición, ¿esta lissta para rev? que trabajes sobre tu master no nos afecta si haces uno a la vez y luego lo actualizas antes de empezar otro, -Nunca tocar tu master, solo actualizarlo con pull (o fetch/merge) Jaja, me paso lo mismo hace un año...#105 Si sos curioso: O TAMBIEN no sé que piensa/pensará el eventual 2do reviewer... |
joaquinelio commentedOct 9, 2020
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 commentedOct 9, 2020
@joaquinelio puede hacer una copia del origen y hacer cherry pick a cada commit cerrar este y crear otro PR |
joaquinelio commentedOct 9, 2020
Revisando el PR, tambien podrias deshacer eso (podes copiar la tarea que tradujiste) y hacer un nuevo pr con "regexp str" solamente |
joaquinelio commentedOct 9, 2020
ah no te habia leido |
dbritto-dev commentedOct 9, 2020
lo de Styles and classes esta en draft |
dbritto-dev commentedOct 9, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@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 commentedOct 9, 2020
"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" No tengo problemas que trabajes sobre tu master mientras no implique trabajo extra como bajar tu rama y hacer cherry pick y otro pr, |
dbritto-dev commentedOct 9, 2020
Lo del cherry pick es sencillo son unos cuantos commits que hay que copiar y ya. |
joaquinelio commentedOct 9, 2020
ah otro coment tuyo que no lei antes de mandar el mio... |
joaquinelio commentedOct 9, 2020
ah, no te conozco, |
joaquinelio commentedOct 9, 2020
quizas sepas mas que yo y lo del master y el mix fue solo un error, te entiendo |
joaquinelio commentedOct 9, 2020
entonces te pediria que me ayudes... si ves mi ultimo pr con 58 archivos modificados (que@vplentinax reviso) |
dbritto-dev commentedOct 9, 2020
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 commentedOct 9, 2020
si podes hacer algo para ayudar, estoy medio quemado mandar pr limpio, cerrar este o modificarlo y si tenes experiencia js no vendria mal una ayuda a las review... |
joaquinelio commentedOct 9, 2020
otra vez... |
dbritto-dev commentedOct 9, 2020
este queda cerrado solo faltaría crear otro PR. Si claro también puedo apoyar a hacer las review. |
No description provided.