- Notifications
You must be signed in to change notification settings - Fork229
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... |
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). |
@joaquinelio puede hacer una copia del origen y hacer cherry pick a cada commit cerrar este y crear otro PR |
Revisando el PR, tambien podrias deshacer eso (podes copiar la tarea que tradujiste) y hacer un nuevo pr con "regexp str" solamente |
ah no te habia leido |
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. |
"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, |
Lo del cherry pick es sencillo son unos cuantos commits que hay que copiar y ya. |
ah otro coment tuyo que no lei antes de mandar el mio... |
ah, no te conozco, |
quizas sepas mas que yo y lo del master y el mix fue solo un error, te entiendo |
entonces te pediria que me ayudes... si ves mi ultimo pr con 58 archivos modificados (que@vplentinax reviso) |
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 |
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... |
otra vez... |
este queda cerrado solo faltaría crear otro PR. Si claro también puedo apoyar a hacer las review. |
No description provided.