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

No se incluye URL para crear PR si Todo OK al crear el repo #67

Open
dato opened this issue Dec 19, 2020 · 0 comments
Open

No se incluye URL para crear PR si Todo OK al crear el repo #67

dato opened this issue Dec 19, 2020 · 0 comments

Comments

@dato
Copy link
Member

dato commented Dec 19, 2020

Si se tiene un Todo OK en la primera entrega del primer TP del repo (por ejemplo, un hash), se termina no incluyendo la URL de creación de pull request.

Eso es porque en ese caso se usa el token de watchenbot en lugar de la Github App:

auth_token = cfg.github_token
installation = gh.get_installation(alu_repo.owner, alu_repo.name)
try:
auth = gh.get_access_token(installation.id)
auth_token = auth.token
except github.UnknownObjectException:
# Probablemente el repositorio no existe todavía. Usamos el antiguo token
# hasta que resolvamos
# https://github.com/PyGithub/PyGithub/issues/1730#issuecomment-739111283.
pass

Con el token de wachenbot falla la creación del checkrun; pero eso no debería impedir incluir la URL (e igualmente se podría usar la Github App una vez se crea el repo, si no se resuelve el comentario de arriba).

dato added a commit to dato/algo2_sistema_entregas that referenced this issue Dec 19, 2020
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

1 participant