Kouhei Sutou
kou****@clear*****
2014年 8月 5日 (火) 22:44:01 JST
須藤です。 In <CAH9Y****@mail*****> "[groonga-dev,02571] Re: Groongaのbuildpackについて" on Tue, 5 Aug 2014 08:20:30 +0900, 杉本涼 <sugry****@gmail*****> wrote: > GitHub APIは、GitHubトークンがなくても使えるみたいです。(取得するだけなら) おぉ、そうなんですね。 > git push したので↓おかしいところがあったらアドバイスしてもらえるとうれしいです。 > > https://github.com/groonga/heroku-buildpack-groonga/compare/1ead240fa8...99f0ee1d15 みてみました。 require "octokit"がないので動かないかなぁと思いました。 そもそも、require "octokit"ができるかというと、できないと思 うんですよね。buildpackが動く環境にoctokit gemがインストール されているとは限らないので。 なので、標準ライブラリーだけで実現しないといけないですねぇ。 あと、気になったのはここらへんです。 * tags[0].name.delete("v")は行儀が悪いなぁ * タグを打ってからHeroku用Groongaをビルドするまでの間に デプロイすると失敗しそう delete("v")の方ですけど、このコードは「v」という文字を全部削 除するっていう意味のコードになるんですが、ここでやりたいこと は「最初のvを削除する」ですよね？必要以上の処理をしようとし ているので行儀が悪いなぁと感じました。 tags[0].name.sub(/\Av/, "") とかの方がより「意図を表したコード」になって適切じゃないかと 思いました。 もうひとつの失敗しそうの方ですが、新しいバージョンの方から Heroku用のビルドがあるかを確認していって、最初に見つかったバー ジョンを使う、みたいにするといいんじゃないかと思いました。 そうすれば、タグを打ったとしてもHeroku用のビルドがなければそ のバージョンはスキップするので失敗しなそうです。 という感じのことを思ったんですが、もう一歩改良してもらえると うれしいです！ -- 須藤 功平 <kou****@clear*****> 株式会社クリアコード <http://www.clear-code.com/> Groongaベースの全文検索システムを総合サポート: http://groonga.org/ja/support/ パッチ採用 - プログラミングが楽しい人向けの採用プロセス: http://www.clear-code.com/recruitment/ コードリーダー育成支援 - 自然とリーダブルコードを書くチームへ: http://www.clear-code.com/services/code-reader/
