Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Closed

Conversation

dbritto-dev
Copy link
Contributor

No description provided.

@javascript-translate-botjavascript-translate-bot requested a review froma teamOctober 7, 2020 14:22
@dbritto-devdbritto-dev changed the titleWIP: Methods of RegExp and StringMethods of RegExp and StringOct 7, 2020
@dbritto-devdbritto-dev marked this pull request as ready for reviewOctober 8, 2020 17:25
@joaquinelio
Copy link
Member

joaquinelio commentedOct 9, 2020
edited
Loading

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
ContributorAuthor

@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
ContributorAuthor

ah no te habia leido
a ver...

lo de Styles and classes esta en draft

@dbritto-dev
Copy link
ContributorAuthor

dbritto-dev commentedOct 9, 2020
edited
Loading

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
ContributorAuthor

"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
ContributorAuthor

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
ContributorAuthor

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.

joaquinelio reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@dbritto-dev@joaquinelio@javascript-translate-bot

[8]ページ先頭

©2009-2025 Movatter.jp