Skip to content

Rama clp#1

Open
Veralucia3007 wants to merge 6 commits intodigitalers2021:mainfrom
PauGramajo:Rama_CLP
Open

Rama clp#1
Veralucia3007 wants to merge 6 commits intodigitalers2021:mainfrom
PauGramajo:Rama_CLP

Conversation

@Veralucia3007
Copy link
Copy Markdown

En el commit #231c7c4 realicé cambios sobre la linea 18 reemplazando el contenido de "¿Que caracterizo a esta aplicación por sobre otras en el mercado?" por una característica en el mercado de la aplicación SPOTIFY.
En la linea 20 modifique "Que la diferencia de la competencia?" por una descripción acerca de la diferencia de la aplicación spotify con la competencia.

@Brienne333
Copy link
Copy Markdown

  • En el commit #74b7eec hice cambios en la línea 1 proporcionando el nombre de la aplicación elegida por el grupo.
  • En línea 6 y 7 proporcioné los datos que completan la ficha técnica.
  • Finalmente en líneas de la 13 a la 15, respondí a las preguntas realizadas sobre quiénes son sus fundadores y los proyectos en los que trabajaron anteriormente de la actual.
  • El commit #bd1b1ab fue resultado de un error en el proceso al tratar de subir la información.

@PauGramajo
Copy link
Copy Markdown

  • En el commit 7bc736a realicé cambios en la línea 23 proporcionando ejemplos de distintas tecnologías utilizadas por la aplicación elegida y de la línea 28 a la 31 agregué la bibliografía consultada para el desarrollo del tp.
  • En el commit 577dd2c hice un pull de lo que había en el repositorio remoto y luego el merge.
  • En el commit d4d0c14 se genera en el repositorio un archivo llamado integrantes.md con el nombre de las integrantes del grupo.

Copy link
Copy Markdown
Contributor

@nuxion nuxion left a comment

Choose a reason for hiding this comment

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

Felicitaciones, esta muy bien el TP.
Una sugerencia respecto a la forma de comentar los commits en el "message".

No suele hacer falta especificar que lineas fueron modificadas ni ser tan especifico respecto al cambio (para eso esta a dispoicion el codigo mismo), pero si quede claro el proposito del cambio, por ejemplo:

git commit -m "Se agregan los links de referencia

- No se encontro link de wikipedia 
- pero se sumaron articulos que salieron en clarin.com"

los mensajes de los commits tienen un titular y pueden tener opcional un cuerpo donde se ahonda en ciertos detalles.

Si quieren ver un ejemplo pueden revisar el historial de este repositorio de la comunidad argentina de python:

Python Argentina ejemplo

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.

4 participants