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

Capítulo de planificación #35

Merged
merged 24 commits into from
Jul 16, 2024
Merged

Capítulo de planificación #35

merged 24 commits into from
Jul 16, 2024

Conversation

danigonzser
Copy link
Owner

@danigonzser danigonzser commented May 6, 2024

Este PR va a ser destinado a realizar el capítulo de planificación. En este capítulo va a aparecer la metodología explicando como se está haciendo el control de calidad del producto basándonos en la diferentes herramientas y metodologías que estamos aplicando al proyecto.

Por ahora #32 ha sido la issue que ha originado esta PR.

@danigonzser
Copy link
Owner Author

Listo para revisión @JJ

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Deberías intentar hacer los PRs un poco más atómicos. El hecho de que hayas quitado tests impide que se apruebe lo que podría ser solo un PR de texto. Quizás deberías dividir en dos partes como antes para que podamos tratar cada uno por separado.

@JJ
Copy link

JJ commented May 17, 2024

Este PR va a ser destinado a realizar el capítulo de planificación. En este capítulo va a aparecer la metodología explicando como se está haciendo el control de calidad del producto basándonos en la diferentes herramientas y metodologías que estamos aplicando al proyecto.

Por ahora #32 ha sido la issue que ha originado esta PR.

* ⚠️ Se ha originado un bug [Bug con macro de LaTeX  \lstset y TeXtidote #36](https://github.com/danigonzser/proyecto-tfg/issues/36) con TeXtidote ya que al utilizar un bloque de código personalizado la macro lstset no la elimina. Se ha hecho un hotfix temporal en [e19ceee](https://github.com/danigonzser/proyecto-tfg/commit/e19ceeeac19ce8ca34a04ab6de2ab1d6d5ef2d8a) y [593ff0e](https://github.com/danigonzser/proyecto-tfg/commit/593ff0ecd715a0ea1fa1122ae4a825aa19e425ea).

Eliminar los tests no es un hotfix. Y en todo caso, debes limitar el ámbito de un PR antes de hacerlo; una cosa son los problemas que tengas en los tests y otros el código que vayas a aportar.

@danigonzser danigonzser reopened this May 21, 2024
@danigonzser
Copy link
Owner Author

@JJ he dividido los PR creo que bien

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Contar qué has decidido no es explicar el proceso de toma de decisiones. Para explicar una decisión (que es una de las cosas que se puntúan en la memoria) hay que

  1. Enmarcar la decisión, decir por qué hay que tomar una decisión en ese área.
  2. Criterios de búsqueda de las posibles soluciones y herramientas. Aquí hay que demostrar que se entiende qué es lo que hay que decidir y qué función ocupa en el proyecto.
  3. Requisitos de la herramienta que te hagan optar por una u otra. Deben ser criterios objetivos.
  4. Cómo cumplen o no esos requisitos cada una de las opciones posibles.

Todo esto no es "metodología", sino infraestructura. Y no puedes titular la sección donde cuentas que se ha elegido una herramienta con la herramienta, porque se supone que lo que tienes que hacer es demostrar madurez para poder elegir entre diferentes opciones en un marco determinado.

@danigonzser
Copy link
Owner Author

Listo para revisión @JJ

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Queda todavía una cierta confusión de conceptos que convendría que aclararas.

@danigonzser
Copy link
Owner Author

Listo para revisión @JJ

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Por favor, limítate a meter en el capítulo de planificación cómo has organizado el proyecto y las herramientas que hayas usado para llevar a cabo esa planificación. El resto lo puedes sacar del PR (y del capítulo), si ves que no tiene relevancia.
Cuando una sección o subsección es irrelevante, no se convierte en relevante porque cambies la cabecera de la misma. Será relevante cuando lo enmarques en el capítulo en el cual el tribunal pueda entender las decisiones tomadas.

@danigonzser
Copy link
Owner Author

He decidido dejar la sección del corrector ortográfico para más adelante. Listo para revisión @JJ

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Por favor, trata de ver qué contenidos son relevantes al capítulo y cuales no, en general Si te limitas a eliminar los que he comentado algo, no estás revisando si el texto corresponde al concepto correspondiente.

@danigonzser
Copy link
Owner Author

Ups no había mencionado. Listo para revisión @JJ

@danigonzser danigonzser added the pending-review Esperando su revisión label Jun 25, 2024
Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

El problema principal es que no has hecho referencia al uso de workflows de github para comprobar automáticamente la calidad, y que has metido más milestones de los razonables. El resto son para tu consideración, pero no bloquean el que se fusione.

@danigonzser
Copy link
Owner Author

Listo para revisión @JJ

@danigonzser danigonzser requested a review from JJ July 6, 2024 11:42
Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Sin meterme en el texto, que ya hemos revisado, te volvería a plantear la necesidad de reconsiderar la colocación del texto en este capítulo específicamente.

Signed-off-by: danigonzser <[email protected]>
@danigonzser
Copy link
Owner Author

Listo para revisión @JJ

Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Deberías intentar que cada uno de los capítulos incluyera lo que describe la cabecera. Si alguna sección no corresponde a la misma, deberías considerar cambiarla de sitio.

@danigonzser
Copy link
Owner Author

Listo para revisión @JJ

@danigonzser danigonzser requested a review from JJ July 11, 2024 19:41
Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

Recuera para los #2 que no conozcan GitHub que de lo que se trata en este capítulo y en los pantallazos es cubrir la rúbrica que indica que hace falta justificar documentalmente las decisiones tomadas y cómo se han seguido efectivamente.

@danigonzser
Copy link
Owner Author

He añadido algunas referencias a las rúbricas @JJ

@danigonzser danigonzser mentioned this pull request Jul 15, 2024
2 tasks
Copy link

@JJ JJ left a comment

Choose a reason for hiding this comment

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

👍

@danigonzser danigonzser merged commit 6aed737 into master Jul 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M00 Corresponde al Milestone 00 pending-review Esperando su revisión
Projects
Status: Fusionado/Cerrado
Development

Successfully merging this pull request may close these issues.

Bug con macro de LaTeX \lstset y TeXtidote No existe una descripción de cómo se va a organizar la memoria
2 participants