Memoize response from `has_rate_limit?` to avoid extra API call

78a5de99 · Douglas Barbosa Alexandre · be9262dd · 78a5de99
Commit 78a5de99 authored Jul 12, 2016 by Douglas Barbosa Alexandre
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

lib/gitlab/github_import/client.rb lib/gitlab/github_import/client.rb +3 -1

No files found.
--- a/lib/gitlab/github_import/client.rb
+++ b/lib/gitlab/github_import/client.rb
@@ -86,7 +86,9 @@ module Gitlab
      end

      def has_rate_limit?
-        rate_limit.present?
+        return @has_rate_limit if defined?(@has_rate_limit)
+
+        @has_rate_limit = rate_limit.present?
      end

      def rate_limit_exceed?