Remove tap and use simplified method call

42a8436a · Oswaldo Ferreira · 890f9801 · 42a8436a
Commit 42a8436a authored Jan 29, 2018 by Oswaldo Ferreira
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

lib/api/entities.rb lib/api/entities.rb +2 -1

No files found.
--- a/lib/api/entities.rb
+++ b/lib/api/entities.rb
@@ -581,7 +581,8 @@ module API
      # See https://gitlab.com/gitlab-org/gitlab-ce/issues/42344 for more
      # information.
      expose :merge_status do |merge_request|
-        merge_request.tap(&:check_if_can_be_merged).merge_status
+        merge_request.check_if_can_be_merged
+        merge_request.merge_status
      end
      expose :diff_head_sha, as: :sha
      expose :merge_commit_sha