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

Identificar y definir gráficas que ayuden a interpretar las revisones de código #5

Closed
clopezno opened this issue Oct 4, 2016 · 5 comments
Labels

Comments

@clopezno
Copy link
Collaborator

clopezno commented Oct 4, 2016

Análizar el concepto de revisión de código extrayendo el diagrama conceptual para definir gráficas que ayuden a analizar el proceso de revisión de código en un proyecto.

conceptualmodelcodereview

Usar ejemplos de de componentes gráficos de la biblioteca que se vaya a utilizar para implementación. Lo dejo a elección de Mario.

@clopezno clopezno added this to the Iniciación milestone Oct 6, 2016
@mjuez mjuez removed this from the Sprint 1 milestone Mar 9, 2017
@mjuez
Copy link
Owner

mjuez commented Mar 31, 2017

Una de las mejoras a incluir en #19, es añadir los conceptos teóricos de este diagrama. Como comentamos en la pasada reunión, la parte importante del diagrama es la entidad "Review", no Revision y Version, ya que se refieren al cambio en sí, además estas dos entidades al parecer no existen en GIT, si no que son típicas de subversion (véase esta pregunta en stackoverflow), en GIT se podría decir que versión y revisión del cambio son lo mismo, y es el identificador del commit.

Estoy de acuerdo con incluir el diagrama en la parte de conceptos teóricos, pero creo que debe ser un diagrama adaptado a lo que tenemos en git, mi propuesta es la siguiente:

nuevo diagrama

@clopezno
Copy link
Collaborator Author

clopezno commented Apr 3, 2017

No puedo ver el contenido del enlace al nuevo diagrama pero estoy totalmente de acuerdo con tu razonamiento.

@mjuez
Copy link
Owner

mjuez commented Apr 3, 2017

Se había roto el enlace a la imagen del diagrama, ahora parece que ya se vuelve a ver.

@clopezno
Copy link
Collaborator Author

clopezno commented Apr 5, 2017

Perdona Mario he contestado en #19.

@mjuez
Copy link
Owner

mjuez commented Apr 5, 2017

Ya he hecho los cambios, creo que podemos dar por cerrada esta issue.

@mjuez mjuez closed this as completed Apr 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants