Skip to content

Pull Request para el Trabajo Practico N°1#5

Open
ElectroGalvanico wants to merge 2 commits intodigitalers2021:mainfrom
ElectroGalvanico:main
Open

Pull Request para el Trabajo Practico N°1#5
ElectroGalvanico wants to merge 2 commits intodigitalers2021:mainfrom
ElectroGalvanico:main

Conversation

@ElectroGalvanico
Copy link
Copy Markdown

Pasa algo raro cada vez que uso las tildes. Mejor no hacerlo. Ojalá esté bien profesor. Tiene un commit por cada uno de los integrantes, excepto el que no pudimos contactar.

BarbaraSuarez291 and others added 2 commits June 1, 2021 16:16
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.

Bien resuelta la parte de realizar los commits, pero falto un poco de informacion principalmente a la parte de tecnologia, por ejemplo de que esta hecho en c++.

Algo que me llamo la atencion de esta app (no la conocia), es que el concepto detras de por que se hizo tenia con ver con generar un "What-You-See-Is-What-You-Get" WYSIWYG para la musica. WYSIWYG parece un acronimo imposible pero esta muy presente en los editores de texto.. algo que seguro usaron pero no sabia que se llamaba asi como por ejemplo Microsoft Word.

Respecto a git, tengan en cuenta que los comentarios pueden ser multiple linea, es decir pueden tener un titulo que resuma la implementacion y un cuerpo que amplie la informacion:

git commit -m "Integrantes y ficha tecnica

- Se agregan los integrantes
- Se modifica la ficha tecnica"

Muy bien e interesante la app.

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.

3 participants