opened
Tamiflu9
username_0
No se si los test estarán bien, pero no me saltan errores..

Al comprobar el pylint me daba un 9.99/10 y he añadido dos cosas para que no me salieran los mensajes de error y conseguir el 10/10. No se si he hecho bien.
Botón para descargar el código de un envío y sus tests
issue
created
emartinm
username_1
Me resulta imposible revisar un PR que dice haber modificado 37 ficheros :-( Sobre todo porque el 95% de los cambios que dice tener no son tuyos, y es una auténtica locura. Otra cosa que tendrías que corregir son los conflictos en lsql/judge/tests/test_views.py.

Creo que algo mal estamos haciendo en la manera en la que actualizáis vuestro fork con los cambios del upstream. Porque antes de hacer el PR actualizas los ficheros de tu repositorio a partir del principal y fusionas los cambios, ¿verdad? ¿Podrías decirme qué comandos ejecutas para actualizar tu repositorio fork con los cambios del repositorio principal?
comment
created
Tamiflu9
username_0
No uso comandos, me descargue una aplicación: Github-Desktop, que parecía más fácil de usar pero igual no lo estoy haciendo bien. He probado ahora a hacerlo con comandos clonando el repositorio otra vez y todo y no me deja actualizar mi repositorio. 
He usado estos comandos:

$ git clone https://github.com/username_0/lsql.git
$ cd lsql
$ git remote add upstream https://github.com/username_1/lsql.git
$ git fetch upstream
$ git rebase upstream/master

-- aqui me da un error el TODO.md

error: could not apply 4e10dcf... Update TODO.md
CONFLICT (content): Merge conflict in TODO.md

y luego a la hora de actualizar no hace nada.
$ git push -f origin master
comment
created
emartinm
username_1
Resolviste al final el conflicto con TODO.md, @username_0 ? Es que ahora me aparece un nuevo commit mucho más manejable, que podría revisar. ¿Hiciste algo concreto?
comment
created
Tamiflu9
username_0
al cambiar el test_view.py y resolver los conflictos me pedía hacer un commit y después de hacerlo ahi si que me parecían solo mis cambios (y no como antes que salían todos los que se habían hecho hasta el momento) y no se si ha sido eso lo que ha cambiado, porque no he hecho nada mas creo...
comment
created
emartinm
username_1
Cerrado porque es muy antiguo y se hará uno nuevo desde cero.
comment