Commit f31709dc authored by Justin Ho's avatar Justin Ho

Clean up code

- Remove unneeded requires
- Use fork URL for comments
parent ac8d1f74
# frozen_string_literal: true # frozen_string_literal: true
require 'slack-messenger'
module ChatMessage module ChatMessage
class BaseMessage class BaseMessage
RELATIVE_LINK_REGEX = /!\[[^\]]*\]\((\/uploads\/[^\)]*)\)/.freeze RELATIVE_LINK_REGEX = /!\[[^\]]*\]\((\/uploads\/[^\)]*)\)/.freeze
......
# frozen_string_literal: true # frozen_string_literal: true
require 'slack-messenger'
module ChatMessage module ChatMessage
class PipelineMessage < BaseMessage class PipelineMessage < BaseMessage
......
...@@ -36,7 +36,7 @@ class SlackService < ChatNotificationService ...@@ -36,7 +36,7 @@ class SlackService < ChatNotificationService
private private
def notify(message, opts) def notify(message, opts)
# See https://github.com/stevenosloan/slack-notifier#custom-http-client # See https://gitlab.com/gitlab-org/slack-notifier/#custom-http-client
notifier = Slack::Messenger.new(webhook, opts.merge(http_client: HTTPClient)) notifier = Slack::Messenger.new(webhook, opts.merge(http_client: HTTPClient))
notifier.ping( notifier.ping(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment