[180726]Gitlab の Merge Request から学ぶ

人のMR(PR)は勉強になると思う毎日。

今日もGitLabの気になった MR を見ていく！

Use limit parameter to retrieve Wikis from Gitaly

メソッド化

こういうのってコードの意図がわかっていいなと思う。
あと、テストでも def でメソッド定義して意図を明確にしとっていいなと思った。

app/controllers/projects/wikis_controller.rb

  def load_project_wiki
-   @project_wiki = ProjectWiki.new(@project, current_user)
+   @project_wiki = load_wiki

    # Call #wiki to make sure the Wiki Repo is initialized
    @project_wiki.wiki

    ...

+ def load_wiki
+   ProjectWiki.new(@project, current_user)
+ end