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

Resolver worktree para ai_corrector.py #56

Open
dato opened this issue Sep 25, 2020 · 1 comment
Open

Resolver worktree para ai_corrector.py #56

dato opened this issue Sep 25, 2020 · 1 comment
Assignees

Comments

@dato
Copy link
Member

dato commented Sep 25, 2020

Hay dirty state (cambios sin commit) en:

turing:/srv/algo2/corrector/repo/ai_corrector.py

Estos, por tanto, no fueron importados al integrar el corrector en entregas. En otras palabras, el dirty state no fue propagado a:

turing:/srv/algo2/entregas/repo/algorw/corrector/ai_corrector.py

Como la primera ruta es el corrector legacy, y la segunda es la versión que verdaderamente se envía como respuesta a la entrega, supongo que quizás a alguien le interese agarrar los dirty changes de la primera ruta, y bien aplicarlos a la segunda ruta, o mejor aún mandar un PR, en el repo de entregas.

(Nótese que el repo de entregas formatea sus archivos con black. Por favor no romper eso, simplemente es cuestión de correr black . antes de hacer commit.)

No es urgente, porque el dirty state prácticamente no molesta.

@mbuchwald
Copy link
Contributor

Creo que los cambios eran nomás de no mandar mensajes que involucren salidas e invitaban a tomar una cervesa, pero siendo que ya está todo habilitado, podría dejarse como en el original y ya está

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

No branches or pull requests

2 participants