Skip to content

chore(post): 50 herramientas cli#129

Merged
Nachichuri merged 25 commits intomasterfrom
50-herramientas-cli
Oct 14, 2025
Merged

chore(post): 50 herramientas cli#129
Nachichuri merged 25 commits intomasterfrom
50-herramientas-cli

Conversation

@axelvf
Copy link
Member

@axelvf axelvf commented Oct 13, 2025

Para revisión final

@axelvf axelvf requested a review from Nachichuri October 13, 2025 16:47
@axelvf
Copy link
Member Author

axelvf commented Oct 13, 2025

@matiasgratz este es el PR final: quedaría ver el tema de la imagen de portada y el título
cc @Nachichuri

@matiasgratz
Copy link
Contributor

Buenísimo le quiero pegar una revisión total más. Con el formato y todo y ya lo sacamos.
El título me convence a mi aunque dude al principio pero está bien la traducción me parece.

@Nachichuri
Copy link
Member

Que complicado cuando ChatGPT traduce los commandos eh xD

@axelvf
Copy link
Member Author

axelvf commented Oct 14, 2025

LOL

Copy link
Member

@Nachichuri Nachichuri left a comment

Choose a reason for hiding this comment

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

Buenas, gracias a ambos por el laburo.

Como feedback para la próxima, en este post en particular faltó mucha revisión previa al envío de el PR, había muchas cosas rotas, en mis commits (salvo el último) pueden ver las cosas que arreglé, estuve +1h

@matiasgratz pudiste levantarte hugo en local? La mayoria de las cosas que arreglé saltan al toque cuando entras al post, como la tabla de contenidos que estaba rota y no andaba (se ve que hugo no acepta refs locales, ademas los refs que estaban no matcheaban los titles), o comandos que estaban traducidos al español y quedaron así, entre otros...

Ya quedo listo, de nuevo gracias por el laburo inicial de subir todo!

@Nachichuri Nachichuri merged commit f76b7d9 into master Oct 14, 2025
1 check passed
@axelvf
Copy link
Member Author

axelvf commented Oct 14, 2025

Gracias @Nachichuri por el feedback.
El tema de los links locales me había funcionado, de hecho los había corregido todos.

No sé si es a mi únicamente, pero ahora no me andan los de la tabla de contenidos.

Te tomo la mejora de la tabla para poner el source, autor y lenguaje. Queda mucho más amigable a la vista.

@Nachichuri
Copy link
Member

Sep @axelvf ahi meti hotfix para arreglar el index, ya quedó fixeado, hay algo raro porque en local estos href no me levantan, capaz estoy manqueando con alguna flag. Gracias!

@axelvf axelvf deleted the 50-herramientas-cli branch October 14, 2025 14:15
@matiasgratz
Copy link
Contributor

Buenas @Nachichuri gracias por el feedback. Si pude levantar hugo hace rato. Tengo varios problemas con markdown, no se como funciona el indice, ni como son algunas de las convenciones en los commits que hiciste.
Me tengo que poner a estudiar eso repasarlo o ir pingponeandolo.
No tuve tiempo de revisarlo a nivel general. Supongo que el disclaimer del principio del autor es algo que tambien se pone al principio.
Me gustaria (y puedo) tomar el articulo antes de nachi y despues de los commits de nachi ahi capaz puedo ver que estaba mal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants