search

ViBii / mitemiru

2 stars 0 forks source link

[IR] Octokitを利用し、mitemiruの中の特定した開発者が自分以外の開発者のissuesへのcomment回数を表示する機能の実装 #174

Closed Altairzym closed 9 years ago

Altairzym commented 9 years ago

What I did:

http://localhost:3000/comments_counter/getcomments

にアクセスすれば、以下の画面が表示される image

レビューの時注意!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

http://localhost:3000/comments_counter/getcomments
Altairzym commented 9 years ago

裏側はまだハードコーディングいているけど、とりあえず自分以外の、かつcommentsは0ではないissuesの情報が全部取れるようになった image この後はこれらのissuesからある開発者(ViBiiの中のAltairzym)はどんなissueにcommentしたか、そのissueの担当者名を表示したい

issue番号 issueタイトル issue担当者名 comment数(Altairzymがこのissueに対するcomment数)

の様な形式で表示したい

参考: http://www.rubydoc.info/github/pengwynn/octokit/Octokit%2FClient%2FIssues%3Alist_issues の中の

Octokit.issue_comments("octokit/octokit.rb", "25")
Altairzym commented 9 years ago

問題が発生した ループで各issueの中のcommentsからある開発者のcommentを洗い出いたい場合、githubにアクセスする制限がある、普通の場合、60回/hですが、認証してから、多分5000回/hになる、しかし将来実用する時5000回/hが足りるかどうか分からない。。。 参考:https://api.github.com/rate_limit @yaginuuu @ryoki1118

yaginuuu commented 9 years ago

@Altairzym 実装部分で問題あれば観ます〜！！

Altairzym commented 9 years ago

@yaginuuu 分かりました、もう少し修正して、もし問題まだあったら、またよろしく！

yaginuuu commented 9 years ago

@Altairzym はいー！

Altairzym commented 9 years ago

@yaginuuu 出てきました！ 今のViBiiの規模を見ると、一回実行したら、大体40回問い合わせをした、ですので5000回は多分大丈夫かな。。。 下の図の中の最後commentsはissueの中のcommentsの総数ですが、 comment回数は私(Altairzym)がcommentした回数です！！ image

yaginuuu commented 9 years ago

@Altairzym なるほどーこれは結構、5000回きついかもなー

yaginuuu commented 9 years ago

@Altairzym 一度にすべてのissueをとってくることって出来るんでしょ？ そしたらもっと減らせないかな？

Altairzym commented 9 years ago

@yaginuuu 現時点は全てのissueではなくて、私が担当しなかった,かつ comment数は0ではない,かつ 担当者がnilではないissueの一覧表示です

yaginuuu commented 9 years ago

@Altairzym であれば、これで問い合わせ1回なんじゃないの？

Altairzym commented 9 years ago

@yaginuuu issue一覧は一回ですけど、各issueに対して、中のcommentsを取得する時、一つのissueは一回問い合わせします

yaginuuu commented 9 years ago

それって一度にとってこれないの？

Altairzym commented 9 years ago

@yaginuuu 前調べたgithubAPIから直接issueとcomment一緒に取ってくる機能がなさそうです。。。 例えばここ https://api.github.com/repos/ViBii/mitemiru/issues

Altairzym commented 9 years ago

@yaginuuu 今Octokit.list_issuesと言うmethodを利用していて、 http://www.rubydoc.info/github/pengwynn/octokit/Octokit%2FClient%2FIssues%3Alist_issues 多分このmethodと https://developer.github.com/v3/issues/#list-issues は同じ機能だと考えた

yaginuuu commented 9 years ago

そうだね。一緒だね。

yaginuuu commented 9 years ago

@ryoki1118 Redmineは制限あったっけか？

yaginuuu commented 9 years ago

GitHubのapi制限は問題だな…

Altairzym commented 9 years ago

@yaginuuu 認証(例えば私のusernameとpw)したら、最大限5000回ですね。。

yaginuuu commented 9 years ago

でもどうしようもない問題なので問い合わせ回数は実装でできるだけしないようにしよう

Altairzym commented 9 years ago

@yaginuuu そうですね、実装の方ができるだけ問い合わせ回数を減らしたいです

Altairzym commented 9 years ago

合計の部分をやりました、今はブラウザ上でこんな感じです(画面の一番下部分統計結果と言う内容の追加だけ) 図の中にはyaginuuuと言う開発者が他の開発者にcommentした回数です 担当者がないissueへのcommentが含まれてないです image

Altairzym commented 9 years ago

あ、やばい、自分のPWが忘れた！！！！

yaginuuu commented 9 years ago

@Altairzym いいんじゃないすかね！

okjsc1118 commented 9 years ago

GUI部分作るのでここからブランチ切ってもいいですかー

Altairzym commented 9 years ago

@ryoki1118 ok!

yaginuuu commented 9 years ago

@ryoki1118 ここからブランチをきるというよりかはこのブランチにコミットしていった方がいいかも！ 本当はロジック部分できたらマージしちゃうのがいいんだと思うけど…

Altairzym commented 9 years ago

@yaginuuu @ryoki1118 ただし、現時点で、沢山pullrequestのAssigneeの部分の情報がないので、合計に含めていないです 例えば https://github.com/ViBii/mitemiru/pull/170 の様なpullrequest、これ大丈夫ですか？

yaginuuu commented 9 years ago

@Altairzym ?? プルリクエストはissueに含まれるけど… ココでは純粋なissueだけでいいのでは？？

Altairzym commented 9 years ago

@yaginuuu Assigneeの部分が担当者があるpullrequestも合計に含めている しかしAssigneeの部分がnullの場合、開発者commentしても、誰にcommentしたか分からない。。。

yaginuuu commented 9 years ago

@Altairzym @ViBii/owners とりあえずこのブランチマージ出来る状態ならマージしたほうがいいんじゃない？？まだ[IR]ではないの？

Altairzym commented 9 years ago

@yaginuuu でも可視化部分まだやってないですので、mergeできますか？

yaginuuu commented 9 years ago

@Altairzym うん。 それでいいんじゃない？

プルリクエストは基本考えなくていいんじゃないでしょうか？？

yaginuuu commented 9 years ago

@Altairzym issueとプルリクのDoneの定義を同じにしてしまわないほうが開発効率良くなると思う. isuueはDoneできないけど、プルリクはレビューもらっていいんじゃない？？

Altairzym commented 9 years ago

@yaginuuu しかしpullrequestと普通のissueをどうやって区別するか分からないです。。。

Altairzym commented 9 years ago

@yaginuuu 普通にissueの情報を全部取ると、その中にpullrequestの情報も一緒にとってくる

yaginuuu commented 9 years ago

@Altairzym だから、合計に含めなくてもよい！おｋ？

Altairzym commented 9 years ago

@yaginuuu あ！なるほど・・・

yaginuuu commented 9 years ago

@Altairzym

プルリクエストのコミュニケーションまでシステムで判断できるようにしたければ、プルリクエストのassigneeもつけるようにしてください

とかhelpに書くしかないですね。

Altairzym commented 9 years ago

@yaginuuu そうですね。。。

Altairzym commented 9 years ago

@yaginuuu @ryoki1118 @sjc1209 レビューお願い、これ、ループがあるので、時間かかると思って、一番上の image を見てからレビューお願いします。 レビューしてからコンフリクトを修正してマージする

yaginuuu commented 9 years ago

@Altairzym コンフリクトなおしちゃってー

Altairzym commented 9 years ago

@yaginuuu お。。。

Altairzym commented 9 years ago

@yaginuuu @ryoki1118 @sjc1209
コンフリクト修正した、よろしく

yaginuuu commented 9 years ago

@Altairzym なんども言ってるけど[IR]になおしてー

Altairzym commented 9 years ago

@yaginuuu 　ああ。。。何度も忘れてしまった。。。。。

Altairzym commented 9 years ago

@yaginuuu なるほど！！！！

Altairzym commented 9 years ago

@yaginuuu 私のパスワードを見ないで～

yaginuuu commented 9 years ago

@Altairzym できれば .env.sampleファイルにも記述してくれると親切かもね！！

Altairzym commented 9 years ago

@yaginuuu そうですね！そこの修正は忘れてしまった！！！！

sjc1209 commented 9 years ago

すみません、今はどこで自分の名前とパスワードを入れますか。