lib/onebox/engine/github_issue_onebox.rb in onebox-1.8.22 vs lib/onebox/engine/github_issue_onebox.rb in onebox-1.8.23

@@ -14,36 +14,35 @@
       end
 
       private
 
       def match
-         @match ||= @url.match(/^http(?:s)?:\/\/(?:www\.)?(?:(?:\w)+\.)?github\.com\/(?<org>.+)\/(?<repo>.+)\/(?<type>issues)\/(?<item_id>[\d]+)/)
+        @match ||= @url.match(/^http(?:s)?:\/\/(?:www\.)?(?:(?:\w)+\.)?github\.com\/(?<org>.+)\/(?<repo>.+)\/(?<type>issues)\/(?<item_id>[\d]+)/)
       end
 
       def data
 
-        @raw ||= ::MultiJson.load(open(url,"Accept"=>"application/vnd.github.v3.text+json",:read_timeout=>timeout )) #custom Accept header so we can get body as text.
-        body_text=  @raw["body_text"]
+        @raw ||= ::MultiJson.load(open(url, "Accept" => "application/vnd.github.v3.text+json", read_timeout: timeout)) #custom Accept header so we can get body as text.
+        body_text = @raw["body_text"]
 
-
-        content_words = body_text.gsub("\n\n","\n").gsub("\n","<br>").split(" ") #one pass of removing double newline, then we change \n to <br> and later on we revert it back to \n this is a workaround to avoid losing newlines after we join it back.
+        content_words = body_text.gsub("\n\n", "\n").gsub("\n", "<br>").split(" ") #one pass of removing double newline, then we change \n to <br> and later on we revert it back to \n this is a workaround to avoid losing newlines after we join it back.
         max_words = 20
         short_content =  content_words[0..max_words].join(" ")
         short_content << "..." if content_words.length > max_words
 
         ulink = URI(link)
-        status_color = {"open"=>"#6cc644","closed"=>"#bd2c00","merged"=>"#6e5494"}
-        result = { link: @url,
-                   title: "Issue: " + @raw["title"],
-                   content: short_content.gsub("<br>","\n"),
-                   labels: @raw["labels"],
-                   user: @raw['user'],
-                   created_at: @raw['created_at'].split("T")[0], #get only date for now
-                   closed_at: (@raw['closed_at'].nil? ? "" : @raw['closed_at'].split("T")[0]),
-                   closed_by: @raw['closed_by'],
-                   avatar: "https://avatars1.githubusercontent.com/u/#{@raw['user']['id']}?v=2&s=96",
-                   domain: "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}",
-                  }
+        {
+          link: @url,
+          title: "Issue: " + @raw["title"],
+          content: short_content.gsub("<br>", "\n"),
+          labels: @raw["labels"],
+          user: @raw['user'],
+          created_at: @raw['created_at'].split("T")[0], #get only date for now
+          closed_at: (@raw['closed_at'].nil? ? "" : @raw['closed_at'].split("T")[0]),
+          closed_by: @raw['closed_by'],
+          avatar: "https://avatars1.githubusercontent.com/u/#{@raw['user']['id']}?v=2&s=96",
+          domain: "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}",
+        }
       end
     end
   end
 end