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

chore(modify readme):Adjust message in the General Considerations sec… #1437

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

AlexisJoselyn
Copy link
Contributor

…tion about User Stories to avoid sounding imperative in PT and SP version

…tion about User Stories to avoid sounding imperative in PT and SP version
@mfdebian mfdebian self-requested a review July 7, 2023 16:23
@mfdebian mfdebian added idea Ideas, sugerencias, comentarios generales y feedback content Relacionado al contenido de proyectos y tópicos translation Cualquier tema relacionado a traducción de textos labels Jul 7, 2023
@mfdebian mfdebian added this to the v6.4 milestone Jul 7, 2023
@mfdebian mfdebian linked an issue Jul 7, 2023 that may be closed by this pull request
@mfdebian
Copy link
Collaborator

mfdebian commented Jul 7, 2023

mil gracias @AlexisJoselyn por levantar el tema y por entregar también la solución 😊 sólo arroja error en los test porque tenemos seteado como pre-test que se ejecute nuestra herramienta mdLint que tiene una regla de que cada línea no debiera exceder los 80 caracteres, en este caso ambos párrafos (en ES y PT) han quedado en una sola línea y por eso arroja el error, pero creo que puedo arreglarlo incluso desde esta interfaz de github, haré eso y lo apruebo ⭐

mil gracias nuevamente! 🙌

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.

Dime que opinas de estos cambios @AlexisJoselyn 😊

la siguiente sin haber completado la anterior. La historia se completa cuando
se cumplen **todos** sus Criterios de Aceptación + **toda** su Definición
de Terminado.
* Sería preferible que se trabaje en una única historia a la vez, para garantizar que cada una se complete de manera adecuada. Es importante asegurarse de cumplir con todos los Criterios de Aceptación + Definición de Terminado antes de avanzar a la siguiente historia. De esta manera, podemos garantizar un trabajo eficiente y de calidad.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* Sería preferible que se trabaje en una única historia a la vez, para garantizar que cada una se complete de manera adecuada. Es importante asegurarse de cumplir con todos los Criterios de Aceptación + Definición de Terminado antes de avanzar a la siguiente historia. De esta manera, podemos garantizar un trabajo eficiente y de calidad.
* Recomendamos fuertemente que trabajen en una única historia a la vez
para garantizar que cada una se complete de manera adecuada.
Es importante asegurarse de cumplir con **todos** los _Criterios de
Aceptación_ + _Definición de Terminado_ antes de avanzar a la siguiente
historia. De esta manera, podemos garantizar un trabajo eficiente y de
calidad.

Modifiqué un poco el texto para encontrar un punto medio entre el mensaje que estaba y el que propones, dime si te hace sentido y lo commiteo 🙏

Copy link
Contributor Author

@AlexisJoselyn AlexisJoselyn Jul 10, 2023

Choose a reason for hiding this comment

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

Sí me hace sentido, gracias @mfdebian así suena más amigable y con la importancia de terminar bien una HU, con esto si tienen dudas lo pueden consultar con sus coaches.
Lo del mdLint lo corrí, pero no identificaba el problema. Gracias por hacerme saber lo de las líneas, lo tendré presente para futuros PR.

Copy link
Member

@unjust unjust Jul 10, 2023

Choose a reason for hiding this comment

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

@mfdebian @AlexisJoselyn En mi opinion me gusta mas "Seria preferible" o "Es recomenable" o solo "recomendamos" - fuertamente no me suena tan flexible.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Agregaría a la conversa a @diegovelezg quien fue quien hizo esos cambios en primer lugar! para que pueda opinar también sobre este cambio 😊 y no "pisemos" su trabajo!

Copy link
Contributor

@diegovelezg diegovelezg Aug 9, 2023

Choose a reason for hiding this comment

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

Creo que no es un asunto de amigable o imperativo (aunque la forma confunda), no tiene nada de malo que se aimperativo si hay una razón de fuerza para hacerlo.

El problema que queremos resolver es que trabajen en una misma cosas por mucho tiempo. El clásico "voy a trabajar en toda la parte de maquetación de todo el propyecto".

¿QUé otras ideas tienen para resolver eso?

Además, si prograsivamente vamos a movernos a "hitos" que con CI (como el piloto de text analyzer de @ssinuco ), ¿cómo converge con eso?

próxima sem completar a anterior. As histórias apenas são concluídas quando se
completam **todos** os critérios de aceitação + **toda** sua definição de
pronto.
* Seria preferível trabalhar em uma única história por vez, a fim de garantir que cada uma seja concluída de maneira adequada. É importante garantir o cumprimento de todos os Critérios de Aceitação e a Definição de Concluído antes de avançar para a próxima história. Dessa forma, podemos garantir um trabalho eficiente e de qualidade.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* Seria preferível trabalhar em uma única história por vez, a fim de garantir que cada uma seja concluída de maneira adequada. É importante garantir o cumprimento de todos os Critérios de Aceitação e a Definição de Concluído antes de avançar para a próxima história. Dessa forma, podemos garantir um trabalho eficiente e de qualidade.
* Recomendamos fortemente que trabalhem em apenas uma história por vez
para garantir que cada uma seja concluída de forma adequada.
É importante certificar-se de cumprir com **todos** os _Critérios de
Aceitação_ + _Definição de pronto_ antes de avançar para a próxima
história. Dessa forma, podemos garantir um trabalho eficiente e de
qualidade.

Lo mismo para PT 👐

@unjust
Copy link
Member

unjust commented Jul 10, 2023

"Fuertamente" no me suena tan flexible...

@mfdebian mfdebian changed the base branch from main to next July 10, 2023 19:44
@unjust unjust modified the milestones: v6.4.0 , Next release Jul 11, 2023
@unjust unjust modified the milestones: 6.5x, next release Jul 21, 2023
@unjust unjust modified the milestones: next release, TBD Aug 29, 2023
@unjust unjust removed this from the TBD milestone Nov 13, 2023
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 idea Ideas, sugerencias, comentarios generales y feedback translation Cualquier tema relacionado a traducción de textos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Content Readme Social Network
4 participants