GeezerVison is showing github.com — Update article.md by JesusZS · Pull Request #385 · javascript-tutorial/es.javascript.info
Conversation
Traducción casi finalizada, faltan pulir detalles en lo referente al uso de términos y la gramática, estoy atento a consejos.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Traducción casi finalizada, faltan pulir detalles en lo referente al uso de términos y la gramática, estoy atento a consejos.
|
All committers have signed the CLA.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hola!
Lo primero que noto son los números de línea, necesitamos que respeten exactamente el original,
facilita la review y ESPECIALMENTE permiten un buen diff con el master ingles para las actualizaciones,
gracias
No miré la traducción, y para glosario me gustaría la colaboración de
que anduvo con esto de los patrones y que es buen reviewesero =)
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias por tu colaboración @MrC0D . Quisiera recomendarte mucha precaución con el tema de las líneas por varias razones ya mencionadas por @joaquinelio :
- Dificulta la sincronización del repositorio
- Dificulta la revisión
A parte de esto, también te sugiero prestar atención a las faltas otrográficas:
- Espacios tras puntos.
- Mayúsculas tras puntos
- Acentos
Hacer bien esto nos permitirá a los reviwers hacer unas revisiones más rápidas y así avanzar más rápido.
Gracias!
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
|
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
Hello @vplentinax
1 que hacemos con este pr? "draft"
nros de linea, ortografia, faltan las tareas
pero no seria dificil arreglarlo, ya pusiste varios acentos Y las tareas las puedo poner como issue
2 qué tanto te molesta "mirar adelante /atrás" ?
"mirar hacia adelante" me suena innecesariamente largo, hasta redundante,
pero rae dice que es tema regional,
si te es apenas raro, prefiero como está
si en cambio te suena a error, lo ponemos a tu española manera (¿y qué dicen los colombianos?).
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
@vplentinax
el branch esta vivo aunque el autor no
la tomo para no desperdiciar trad y tus correcciones, y le agrego la task
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
@vplentinax
el branch esta vivo aunque el autor no
la tomo para no desperdiciar trad y tus correcciones, y le agrego la task
Si puedes hacerte cargo estaría bien...
Por cierto, espero que el autor también esté vivo!
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
joaquinelio
commented
Feb 16, 2021
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
|
Ugh.
no quise matarlo...
tengo que "cuidar mi lenguaje"
hiciste montones de correcciones, las acepte todas,
resolvi 2 conflictos....
en un art de poco más de 100 lineas jajaj, los coment son mas largos
todo por respetar los 2 autores
tengo problems con lookahead lookbehind. No es una palabra, es una contraccion para manejarse mejor
en ruso le puso "anticipatorio retrospectiva
"mirar hacia delante" me suena feo, largo y redundante pero bue, DRAE dice que es regional y lo acepte
pero es impractico
mira esto
También podemos unirlos en un solo lookbehind aquí:
"en un solo mirar hacia delante aqui: " ??
y hay frases con varios lookahead /behind
quiza invente
verdelante
no se que poner
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
joaquinelio
commented
Feb 18, 2021
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
|
@vplentinax
antes de que corrijas explico motivos de los mas polemicos
-
linea 3 preceder y anteceder son sinonimos ( es "precede / sucede" pero "sucede" confundiria)
-
en unix usabamos "look-ahead" incluso en traducciones, cortito y practico. Lo hubiera dejado ingles.
Como no es un comando ni palabra clave prefiero "ver delante" asi entre comillas (porque significa: "revisar lo que tenés delante")
Lo veo importante para no embarrar frases con varios "mirar hacia delante", es imposible no distraerse.
Ah, "ver delante" es correcto en España y latam, (www.fundeu.es/recomendacion/adelante/) aunque "coloquiemos" distinto.
Pero
en MDN aparece como sugeriste "mirar hacia delante", no suelo hacerle caso porque su Esp suele ser incoherente... aunque no acá.
Más que gramática me encantaria tener opinion de @cortizg u otro experimentado acá, pero si no, aceptaría tu veto.
-
36 esta mal, la reescribi
y la reescribí bastante distinto en PR ingles (seguro Ilya rechaza pero deberia corregirlo de algun modo)
-
Debería haber corregido el codigo y vuelto los string de nuevo al ingles, no lo hice. Pereza.
-
si aceptas "ver delante" sigo con los ejercicios en otro PR, tienen muchos "lookahead"
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
No puedo respetar a MDN. En una explicación ponen
"mirar hacia delante"
"mirar hacia atras"
pero depues
"aserción de búsqueda anticipada "
"aserción de búsqueda inversa "
que no tienen sentido,
ademas "anticipo" es para tiempo, no posición.
correcto:
de subsecuencia
de precedencia
comodo:
derecha
izquierda
o
posterior
anterior
pero sigo con "ver delante - ver detras" salvo que lo vetes, te sigo.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
art cortisimo, pero mucho trbjo de 3 personas
MERGE
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
joaquinelio
added a commit
that referenced
this pull request
Jun 18, 2021
9-14 lookaround tareas -atado otro pr -resolver #385 primero- x temas de glosario
Reviewers
vplentinax
Awaiting requested review from vplentinax
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
|