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

Cambios a Cipher readme #1233 #1255 #1237

Merged
merged 11 commits into from
Oct 18, 2022
Merged

Conversation

unjust
Copy link
Contributor

@unjust unjust commented Oct 3, 2022

Llevamos los cambios que hicimos en DEV001 aca. (Addresses #1233)
Aqui puede ver el README.

Un resumen de los cambios:

Otros cambios que queremos proponer en esta rama:

  • Agregamos algun recurso de objetos y metódos. No hay!
  • Estandarizar este readme y de Card Validation (el parte de descripción de archivos y boilerplate)

@unjust unjust self-assigned this Oct 3, 2022
@msantelices
Copy link
Contributor

En cuanto a objetivos generales, siguiendo la estructura que se implementó para Social Network, los objetivos generales podrían quedar redactados de una forma similar a esta:

  • Trabajar en base a un boilerplate, la estructura básica de un proyecto en distintas carpetas (a través modulos en JS).
  • Conocer las herramientas de mantenimiento y mejora del código (linters y pruebas unitarias).
  • Aprender sobre objetos, estructuras, métodos e iteración (bucles) en JavaScript
  • Implementar control de versiones con git (y la plataforma github)

¿Qué les parece? ¿Alguna sugerencia o mejora?

@ssinuco
Copy link
Collaborator

ssinuco commented Oct 4, 2022

Otros cambios que queremos proponer en esta rama:

* Agregamos algun recurso de objetos y metódos. 

¿Que les parece estos recursos sobre objetos y metodos?:

https://es.javascript.info/object

https://es.javascript.info/object-methods

@mfdebian
Copy link
Collaborator

mfdebian commented Oct 4, 2022

¡Mil gracias @unjust tu liderazgo en esto se agradece mucho! 🙌 💪

Estoy muy de acuerdo con el comentario de @msantelices; Creo que debemos buscar siempre esa "estandarización" entre los READMEs de los proyectos, y como comenta él, en el de social-network hemos escogido usar cierto lenguaje y yo optaría por seguir eso, inclusive le cambiaría el título de "Objetivos de aprendizaje generales" a "Los objetivos generales de este proyecto son los siguientes" y luego enumerarlos como comenta Mauro.

Por otro lado mi opinión personal es la de que las secciones de preámbulo y resúmen del proyecto deberían ir siempre al comienzo 🙈 siento que hace más sentido semántico.

Revisé los links de @ssinuco y el primero (este) me gustó mucho, el segundo siento que hace demasiado énfasis en this 🤔 no sé qué opinan, quizás en vez de ese podemos buscar otro, les parece? 😊

@unjust
Copy link
Contributor Author

unjust commented Oct 5, 2022

@msantelices estoy de acuerdo. Voy a aplicar los cambios
@ssinuco y si! me encanta este recurso y en verdad es que tuve en mente 👏

@moniyama
Copy link
Member

moniyama commented Oct 5, 2022

Pienso que el topic 5 Hito 1 podría ser solo Hito...
Y las antiguas Hacker Editions en topic 6 (Hito 2) podría ser solo Hito opcional/Hitos opcionales..
Así cuándo tuvierem más hitos (tanto normal como las opcionales), las ponemos como subtopics...
Hito 1, 2, 3... (si hacen parte de los criterios de aceptación mínimos del proyecto) y los subtopics opcionales creo q no hay necesidad de numerarlas

BTW me gustaran muchísimo los cambios propuestos

@lupomontero lupomontero added enhancement New feature or request content Relacionado al contenido de proyectos y tópicos labels Oct 12, 2022
lupomontero
lupomontero previously approved these changes Oct 12, 2022
Copy link
Member

@lupomontero lupomontero left a comment

Choose a reason for hiding this comment

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

LGTM ⭐

Solo faltaria validar traducción al portugués y corregir warnings/errores de mdlint

Copy link
Member

@moniyama moniyama left a comment

Choose a reason for hiding this comment

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

Gracias @unjust!
Hice algunas pequeñas sugerencias en la traducción!

projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
projects/01-cipher/README.pt-BR.md Outdated Show resolved Hide resolved
@unjust
Copy link
Contributor Author

unjust commented Oct 14, 2022

Obrigado @moniyama pelos comentários e @gabrieluizramos pela vista. Vou aplicar as alterações. Bons detalhes!

Co-authored-by: Monica Yamazaki <52083679+moniyama@users.noreply.github.com>
Copy link
Collaborator

@mfdebian mfdebian left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

muchísimas gracias @unjust 🤘

gracias a lxs contribuidorxs/reviewers @msantelices @moniyama @gabrieluizramos y a @diegovelezg por impulsar el cambio ♻️

@unjust unjust merged commit 8426814 into Laboratoria:main Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Relacionado al contenido de proyectos y tópicos enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Actualizamos readme de cipher con nuevos constraints
7 participants