Skip to content
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

sync 0825 de dos años #365

Merged
merged 2,187 commits into from
Sep 17, 2020
Merged

sync 0825 de dos años #365

merged 2,187 commits into from
Sep 17, 2020

Conversation

joaquinelio
Copy link
Member

No description provided.

Logan-Schelly and others added 30 commits May 21, 2020 14:10
The task has a test with a missing comma.  That same comma is missing in the comment, so I'm putting it here as well.
The empty users array was undeclared in the second test, and the object in the first test was missing a comma in its declaration.
correct minor error: "имя" is used in place of "name"
Fixed typos on last Array Methods task.
Cross-origin requests initiated by embedded images and forms actually bring cookies by default. Though it is incorrect to say that all cross-origin requests do not bring credentials by default.
Details on cross-origin requests & credentials
@joaquinelio joaquinelio marked this pull request as ready for review August 31, 2020 09:01
Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parece todo correcto

@joaquinelio
Copy link
Member Author

Hola Vale,
veo que ressolviste conflicto que salto con los ultimos añadidos...
(y veo que lo ensuciaron con faltas de ortografía, jeje hubieran dejado a skywalker ya revisado)
(ah, y una linea del rabbit que el eng habia borrado)

Te recomendaria que hicieras merge de este pr,
asi evitar los eternos conflictos que seguirian apareciendo.

y vemos como queda el proximo pr del bot...
Y OJO a los PR del bot, hay que revisar todos los cambios que sugiere, no solo los conflictos.
podes hacer un issue conel new procedim de sync

Todo por el procedimiento que sea legado al repo.
Los proximos sync serán mucho mas fáciles
y los errores y las ¿como es la palabra para la cosas que no ves y dejas de hacer porque no ves? Ando aussente de palabras...
bue, los errores caerian a casi cero y la actualización seria inmediata.
fin a horas de tu escrutinio minucioso
y si aparece tomass pedile diculpass d mi parte

de veras, para terminar de una vez con los conflict de pr, recominendo merge

ta luego, partner.

@joaquinelio
Copy link
Member Author

joaquinelio commented Sep 1, 2020

articulo de tscandalita types

pista: su pr original paso lo mismo, figura que
borró todas las lineass y las escribio todas de nuevo, igual qe aca
git no reconocio ninguna original

es porque su archivo fue grabado formato windows y no unix, con un carácter extra por linea:
hexa 0d0a (retorno del cabezal de impresion al inicio, step motor del papel avanza una linea ja!)
en lugar del unix:
hexa 0a
no lo habia notado porque vs code no pregunto, simplemente lo grabo como cree que debe

y por eso lens no mostraba diferencias, las detectaba pero no tenia forma de mostrármelas.

Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
@joaquinelio
Copy link
Member Author

/done

@vplentinax
Copy link
Contributor

@joaquinelio sobre hacer un new issue con el procedimiento, preferiría que lo hicieras tu ya que comprendiste mejor como hacerlo, así lo tenemos ya para cualquier maintainer que quiera hacer la sync. Eso sí, intenta ser conciso, con los comandos claros, sin comentarios innecesarios. 👍

@joaquinelio
Copy link
Member Author

Veremos.
Es que no pensaba liberarte delas sync, ayer creia que esta sync iba ser mi ultimmo buen aporte, alliviarte lo arduo y mejorar el resultado. Supongo que funcionara.

@vplentinax
Copy link
Contributor

@EzequielCaste continuarás la revisión o hacemos merge¿?

@joaquinelio
Copy link
Member Author

Esto es enfermo,
cada merge debo resolver 3 o 4 conflictos...

fueron 415 archivos!

y si se le ocurre mover un archivo tengo que bajarlo a local de nuevo!!!

@vplentinax ¿podes hacer el merge?
Te digo a vos porque nunca volvi atras un commit...
No , no es "por las dudas", es para guardar algo de profesionalismo...
impossible que salga mal, pero en cuanto lo vea me pongo a revisar el sitio entero... por las dudas jajajja

y quiero ver ssi efectivamente git empieza a hacer track de nuetros articulos y sacarnos el problema de los sync de encima.
Que el issue #348 haya valido de algo!
ni hablar de los 400+ archivos revisados

pff pensaba que esto iba a ser mi legado, cuando meti a @maksumi fue porque queria tomarme vacaciones de repo...

Eh! por qué no merge no esta verde???

@vplentinax vplentinax merged commit 1ff2be8 into javascript-tutorial:master Sep 17, 2020
@vplentinax
Copy link
Contributor

Hecho @joaquinelio ahora echa un repaso completo y comprueba que no se haya roto nada!

@joaquinelio
Copy link
Member Author

@vplentinax
so far so good...

problema (no peor que antes):
Fui perezoso. Explico:
Eran más de 400 archivos, de esos

  • ingles ingles, obvio lo deje hacer.
    -Varios decidí que no requerían cambios, le metí OURS.
    ej, marqué todos lo svg como propios. No se ven acá, no se COMO ni SI lo registra (lo veremos en el proximo PR del bot)
  • resolvi todas las tareas y soluciones, porque eran mas o menos cortas.
    De los articles traducidos...
  • sin conflictos reales, los marqué todos como propios (no aparecen aca pero algunos se hubieran sobrescrito con ingles si no lo evitaba, mecanismo explicado en el pr Pull from English stream overwrites some translated files like if there weren't conflict #348
  • conflictos REALES (cambios en lineas particulares) los hice, son los que aparecen acá, a veces un par, a veces varios en un mismo art.
  • conflicto TOTAL: Un diff completamente diff. Eran 75, NO me puse a verlos y le di OURS sin asco. Hice un log de conflictos pero no disctiminé cuáless son los 75 que requieren revision (no implica que requieran ser actualizados).
    Que tambien sean 75 los modificados aca es casualidad.

Es decir, 75 articles.md, no identificados, saltaron la comprobacion. Igual no me arrepiento, estamos mejor que antes.

Por ejemplo 1-2-15 function basics tiene imagenes png y habria que sacarlas.

Aparte de esso. Queda un trabajo:
Revisar el próximo PR del bot, ver si se redujo... Y VER si es seguro. Sospecho que no es lo mejor como procedimiento, intuición nomas.

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.

None yet