libreim / apuntesDGIIM

Repositorio de apuntes del Doble Grado en Ingeniería Informática y Matemáticas de la Universidad de Granada.
https://libreim.github.io/apuntesDGIIM/
Other
81 stars 26 forks source link

Continuación LMD y PDOO #94

Closed pab1s closed 3 years ago

mariogmarq commented 3 years ago

Le he estado echando un vistazo y esta bastante bien. Lo unico que en los estados de los temas usas el mismo indicador para tema completo que para tema pendiente de revision. Yo lo cambiaria por un emoji de este estilo ❓. A parte de eso, creo que se te han colado otros commits en la PR.

yabirgb commented 3 years ago

Hay conflictos que tendrías que revisar antes de poder mergear. Los comentarios de @mariogmarq creo que no los has actualizado.

pab1s commented 3 years ago

Lo que me ha comentado Mario lo he solucionado creando un Pull Request para la plantilla del README con su rama por separado. Por otro lado, he cambiado el motivo de este pull request para actualizar los apuntes de PDOO y LMD. Sin embargo, me da conflicto el contenido nuevo que he añadido, a pesar de que nadie ha modificado estos apuntes desde la última vez que los actualicé y no entiendo muy bien por qué. ¿Alguien podría echarme una mano con esto?

danipozo commented 3 years ago

Sin embargo, me da conflicto el contenido nuevo que he añadido, a pesar de que nadie ha modificado estos apuntes desde la última vez que los actualicé y no entiendo muy bien por qué. ¿Alguien podría echarme una mano con esto?

No estoy seguro de por qué da conflicto en esos ficheros. En todo caso, se podría mezclar a mano. El problema que veo es que entre los ficheros cambiados sigue habiendo muchos que no son de PDOO ni LMD. ¿Quizá es por cómo hemos mezclado los anteriores PR? Mediante rebase, creo.

Puedes probar a mezclar la rama de libreim en la tuya y ver cómo se actualiza este PR.

pab1s commented 3 years ago

Puedes probar a mezclar la rama de libreim en la tuya y ver cómo se actualiza este PR.

Acabo de hacerlo y ya está solucionado. Se ve que cambié algunos apartados después y por eso me daba conflicto. ¡Gracias!