Pygithub: Aucun commentaire renvoyé pour PullRequest

Créé le 21 août 2012  ·  7Commentaires  ·  Source: PyGithub/PyGithub

Bonjour, voici le problème.
Aucun commentaire retourné. Mais les commentaires comptent pour montrer la valeur appropriée.

p = z. tire[1]
p
PRODUCTION:
SORTIE : u'\u041d\u0435 \u043f\u0440\u0438\u043d\u0438\u043c\u0430\u0442\u044c =)'
p.commentaires
SORTIE : 1
p.get_comments()
PRODUCTION:
liste(p.get_comments())
PRODUCTION: []

Des idées ?

bug feature request

Tous les 7 commentaires

Bonjour!

Je confirme que le code suivant présente le même problème :

import github

g = github.Github()
r = g.get_user( "jacquev6" ).get_repo( "PyGithub" )
p = r.get_pull( 57 )
print p.title, "has", p.comments, "comments"
print [ c.body[ :15 ] for c in p.get_comments() ]

Il imprime :

Allows connection to GitHub Enterprise installs on local URLs has 2 comments
[]

Cette dernière ligne appelle l'API https://api.github.com/repos/jacquev6/PyGithub/pulls/57/comments , comme documenté dans http://developer.github.com/v3/pulls/comments/#list -comments -on-a-pull-request, mais l'API renvoie une liste vide. Je viens de réussir à obtenir les commentaires en appelant plutôt https://api.github.com/repos/jacquev6/PyGithub/issues/57/comments . Le problème doit être du côté de Github.

En guise de solution de contournement

print [ c.body[ :15 ] for c in r.get_issue( p.number ).get_comments() ]

Pour le moment, je n'ai pas assez de temps pour m'occuper de ce problème, mais s'il est toujours là début septembre, je contacterai Github et/ou patcherai PyGithub.

Prendre plaisir,

Pour améliorer un peu la solution de contournement, vous pouvez le faire pour déterminer si un problème est un PR ou non (car tous les PR sont aussi des problèmes) :

        if myIssue.pull_request.diff_url:
            PR=True
        else:
            PR=False

Merci! L'utilisera comme solution de contournement.

Je viens de contacter Github pour ce problème.

Et voici la réponse que j'ai reçue de Github :

From: Wynn Netherland
Subject: API v3, list comments on pull requests

Hi, Vincent. I know it's confusing but we actually have three types of comments on GitHub, so there are
three different spots in the API to grab them. Be sure you're looking in the right spot for the data you expect.

Pull Request comments are the top-level comments found on the Pull Request page. These are retrieved via
the Issues API [1] since PRs are essentially specialized Issues. Pull Request review comments are those made
against the diff on the PR. You can grab these with the Review Comments API [2]. Finally, line comments made
outside the context of a PR on a raw commit can be retrieved via the Commit Comments API [3].

Armed with that info, let me know if you're not seeing the data you expect and the API call you're making, and I
can dig a bit deeper.

[1] http://developer.github.com/v3/issues/comments/
[2] http://developer.github.com/v3/pulls/comments/
[3] http://developer.github.com/v3/repos/comments/

Cheers,

Donc, cela est cohérent avec ce que nous pouvons voir ici : https://github.com/jacquev6/PyGithub/pull/57 , car PullRequest.get_comments utilise l'API Pull Requests. @ nixoz2k7 Est-ce également cohérent avec votre problème d'origine ?

Quoi qu'il en soit, cela est trompeur, j'ajouterai donc deux méthodes nommées get_issue_comments (en utilisant l'API Issues) et get_review_comments (synonyme de get_comments , en utilisant l'API Pull Requests). Attendez-vous à cela pour le milieu de la semaine prochaine.

Je viens de pousser les nouvelles méthodes sur https://github.com/jacquev6/PyGithub/tree/develop. Je ferme le sujet car je pense qu'il est résolu. @nixoz2k7 , n'hésitez pas à le rouvrir si vous avez toujours un problème avec ce qui a été dit.

Merci beaucoup ;)
d'ailleurs, cela fonctionne très bien avec la solution de contournement des "problèmes". stable depuis 2 semaines déjà.

Merci encore.
Sergueï.

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

thefunkjunky picture thefunkjunky  ·  11Commentaires

JosephTLyons picture JosephTLyons  ·  43Commentaires

mattroid picture mattroid  ·  12Commentaires

pconrad picture pconrad  ·  14Commentaires

jayfk picture jayfk  ·  11Commentaires