Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
GitLab FOSS
GitLab FOSS
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Security & Compliance
    • Security & Compliance
    • Dependency List
    • License Compliance
  • Packages
    • Packages
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Code Review
    • Insights
    • Issues
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • GitLab.org
  • GitLab FOSSGitLab FOSS
  • Merge Requests
  • !22684

You need to sign in or sign up before continuing.
Merged
Opened Oct 30, 2018 by blackst0ne@blackst0ne3 of 8 tasks completed3/8 tasks
  • Report abuse
Report abuse

Add Discord integration

  • Overview 67
  • Commits 13
  • Pipelines 12
  • Changes 15

What does this MR do?

Adds a news service - integration with Discord

What are the relevant issue numbers?

Closes #21635 (closed)

Why using fork?

This MR brings the usage of discordrb-webhooks-blackst0ne - a fork of mine. Why do we need to use the fork instead of the upstream version of the gem?

The reason is in dependency conflicts.

The upstream discordrb gem requires 'rest-client >= 2.1.0.rc1' At the same time kubeclient we already use requires 'rest-client ~> 2.0' which raises dependency conflicts.

I have forked discordrb and relaxed the dependency of rest-client. Now it requires 'rest-client ~> 2.0' - the same version as kubeclient expects.

The dependency of rest-client >= 2.1.0.rc1 was added to fix some bugs for Ruby 2.5 on Windows OSes. So it seems we can just relax dependencies and use the gem.

This fork should be replaced by the upstream version when both discordrb and kubeclient start requiring the same version of rest-client.

Does this MR meet the acceptance criteria?

  • Changelog entry added, if necessary
  • Documentation created/updated
  • Tests added for this feature/bug
  • Conforms to the code review guidelines
  • Conforms to the merge request performance guidelines
  • Conforms to the style guides
  • Conforms to the database guides
  • Link to e2e tests MR added if this MR has Requires e2e tests label. See the Test Planning Process.
Edited Nov 05, 2018 by blackst0ne
Assignee
Assign to
11.6
Milestone
11.6
Assign milestone
Time tracking
6
Labels
Community contribution Plan [DEPRECATED] devopsplan documentation external services feature
Assign labels
  • View project labels
Reference: gitlab-org/gitlab-foss!22684

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.