peoplegroup-connectors 0.1.88 → 0.1.89

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5438b021525494c7cf68386c73234d714d661d8bb97d4da6805aebbcabace18
-  data.tar.gz: 61269cad8f934e828b527f68559a633efa7ecdd17ac2055d2f5284333db519ce
+  metadata.gz: 6c4414683f3b6d3b3e9566ba33df66018642cf551ec836e27c00b78b4dee16ee
+  data.tar.gz: 27be0aff4e7d971faaece5e7a984158bd91731037b1614596f98f42ab634b425
 SHA512:
-  metadata.gz: 6157518930851bccf1eedf3f06bae1d405e680c2da947d9286d2f8fb9a0abb8543363b0a630f5e37eab9442f1b521bd6d93d4be8d2aeacd2328bee8e4460f663
-  data.tar.gz: 2a8c60f99d54a5e081532e4645c63b5a1726e5b3a91043a8027e9f5e54798ad95a33207d1d61175ede6d47d024e46e80e765949156dd6d4878713b958e90eaa9
+  metadata.gz: 3bf61018687ad168f08695cf6e971b647735438488d5b8fbbc6a2e6fa69b9763bb1484dfe464b944d9db98b6adb19b0898a78e1cf9872877ae43fd706ba947eb
+  data.tar.gz: 48a97e0dfe60996cf5bcd1fd543c36c9a3429079541d10353031151143d3c0fea0d61584da9e6f2cc8b1213ab0700ed9332df6d4ebf71c95db10f3026edf7a59

data/lib/peoplegroup/connectors/gitlab.rb

@@ -55,6 +55,11 @@ module PeopleGroup
         retry_on_error { @client.remove_group_member(group_id, user_id) }
       end
 
+      def group_member_ids(group_id)
+        members = retry_on_error { @client.group_members(group_id) }
+        members.map { |member| member[:id] }
+      end
+
       def create_issue(project, title, options = {})
         retry_on_error { @client.create_issue(project, title, options) }
       end
@@ -91,11 +96,12 @@ module PeopleGroup
         epic = find_epic(group_id, title)
         reopen_epic(epic) if epic && epic.state == 'closed'
         options[:confidential] = true
-        epic || retry_on_error { @client.create_epic(group_id, title, options) }
+        epic || create_epic(group_id, title, options)
       end
 
-      def commit_new_files_to_new_merge_request(project_id, branch_name, new_files, commit_message, description = nil, target_branch: 'master')
-        retry_on_error { @client.create_branch(project_id, branch_name, target_branch) }
+      def commit_new_files_to_new_merge_request(project_id, branch_name, new_files, commit_message, description = nil, target_branch: 'master', assign_group: nil)
+        create_branch(project_id, branch_name, target_branch)
+
         actions = []
         new_files.each do |file|
           actions << {
@@ -105,45 +111,58 @@ module PeopleGroup
           }
         end
 
-        retry_on_error { @client.create_commit(project_id, branch_name, commit_message, actions) }
+        create_commit(project_id, branch_name, commit_message, actions)
+
+        options = {
+          source_branch: branch_name,
+          target_branch: target_branch,
+          remove_source_branch: true
+        }
 
-        options = { source_branch: branch_name, target_branch: target_branch, remove_source_branch: true }
         options[:description] = description if description
-        retry_on_error { @client.create_merge_request(project_id, commit_message, options) }
+        options[:assignee_ids] = group_member_ids(assign_group) if assign_group
+
+        create_merge_request(project_id, commit_message, options)
       end
 
-      def commit_change_to_new_merge_request_v2(project_id, branch_name, commit_message, description = nil, files_to_delete = [], files_to_update = [], target_branch: 'master')
+      def commit_change_to_new_merge_request_v2(project_id, branch_name, commit_message, description = nil, files_to_delete = [], files_to_update = [], target_branch: 'master', assign_group: nil)
         actions = []
 
         files_to_delete.each do |file|
-          action = {
+          actions << {
             action: 'delete',
             file_path: file
           }
-          actions << action
         end
 
         files_to_update.each do |file|
-          action = {
+          actions << {
             action: 'update',
             file_path: file[:file_path],
             content: File.read(file[:tmp_file_path])
           }
-          actions << action
         end
 
         return unless actions.any?
 
-        retry_on_error { @client.create_branch(project_id, branch_name, target_branch) }
-        retry_on_error { @client.create_commit(project_id, branch_name, commit_message, actions) }
+        create_branch(project_id, branch_name, target_branch)
+        create_commit(project_id, branch_name, commit_message, actions)
+
+        options = {
+          source_branch: branch_name,
+          target_branch: target_branch,
+          remove_source_branch: true
+        }
 
-        options = { source_branch: branch_name, target_branch: target_branch, remove_source_branch: true }
         options[:description] = description if description
-        retry_on_error { @client.create_merge_request(project_id, commit_message, options) }
+        options[:assignee_ids] = group_member_ids(assign_group) if assign_group
+
+        create_merge_request(project_id, commit_message, options)
       end
 
-      def commit_change_to_new_merge_request(project_id, branch_name, file_path, file_with_change, commit_message, description = nil, target_branch: 'master')
-        retry_on_error { @client.create_branch(project_id, branch_name, target_branch) }
+      def commit_change_to_new_merge_request(project_id, branch_name, file_path, file_with_change, commit_message, description = nil, target_branch: 'master', assign_group: nil)
+        create_branch(project_id, branch_name, target_branch)
+
         actions = [
           {
             action: 'update',
@@ -151,11 +170,23 @@ module PeopleGroup
             content: file_with_change
           }
         ]
-        retry_on_error { @client.create_commit(project_id, branch_name, commit_message, actions) }
 
-        options = { source_branch: branch_name, target_branch: target_branch, remove_source_branch: true }
+        create_commit(project_id, branch_name, commit_message, actions)
+
+        options = {
+          source_branch: branch_name,
+          target_branch: target_branch,
+          remove_source_branch: true
+        }
+
         options[:description] = description if description
-        retry_on_error { @client.create_merge_request(project_id, commit_message, options) }
+        options[:assignee_ids] = group_member_ids(assign_group) if assign_group
+
+        create_merge_request(project_id, commit_message, options)
+      end
+
+      def create_epic(group_id, title, options)
+        retry_on_error { @client.create_epic(group_id, title, options) }
       end
 
       def create_epic_note(group_id, epic_id, text)
@@ -188,7 +219,7 @@ module PeopleGroup
         epics.first
       end
 
-      def create_branch(project_id, branch_name, target_branch: 'master')
+      def create_branch(project_id, branch_name, target_branch = 'master')
         retry_on_error { @client.create_branch(project_id, branch_name, target_branch) }
       end
 
@@ -197,7 +228,7 @@ module PeopleGroup
       end
 
       def create_commit(project_id, branch, commit_message, actions)
-        retry_on_error { @client.create_commit(project_id, branch, commit_message, actions, {}) }
+        retry_on_error { @client.create_commit(project_id, branch, commit_message, actions) }
       end
 
       def reopen_epic(epic)

data/lib/peoplegroup/connectors/version.rb

@@ -2,6 +2,6 @@
 
 module PeopleGroup
   module Connectors
-    VERSION = '0.1.88'
+    VERSION = '0.1.89'
   end
 end

data/lib/peoplegroup/connectors.rb

@@ -1,15 +1,8 @@
 # frozen_string_literal: true
 
-require_relative 'connectors/version'
-require_relative 'connectors/gitlab'
-require_relative 'connectors/google_sheets'
-require_relative 'connectors/slack'
-require_relative 'connectors/pto_roots'
-require_relative 'connectors/bamboo'
-require_relative 'connectors/greenhouse'
-require_relative 'connectors/workday'
-require_relative 'connectors/hris'
-require_relative 'utils'
+require 'require_all'
+require_rel 'utils'
+require_rel 'connectors'
 
 module PeopleGroup
   module Connectors

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: peoplegroup-connectors
 version: !ruby/object:Gem::Version
-  version: 0.1.88
+  version: 0.1.89
 platform: ruby
 authors:
 - lien van den steen
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-22 00:00:00.000000000 Z
+date: 2022-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gitlab
@@ -108,6 +108,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.12.1
+- !ruby/object:Gem::Dependency
+  name: require_all
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -128,28 +142,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.91.1
+        version: '0.91'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.91.1
+        version: '0.91'
 - !ruby/object:Gem::Dependency
   name: gitlab-styles
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: '6.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '6.0'
+        version: '6.6'
 - !ruby/object:Gem::Dependency
   name: rubocop-packaging
   requirement: !ruby/object:Gem::Requirement
@@ -170,14 +184,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.39'
+        version: '1.44'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.44'
+- !ruby/object:Gem::Dependency
+  name: dotenv
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.39'
+        version: '2.7'
 description: Avoid repeating methods in different projects for our connectos.
 email:
 - lvandensteen@gitlab.com
@@ -206,7 +234,7 @@ metadata:
   homepage_uri: https://gitlab.com/gitlab-com/people-group/peopleops-eng/connectors-gem
   source_code_uri: https://gitlab.com/gitlab-com/people-group/peopleops-eng/connectors-gem
   changelog_uri: https://gitlab.com/gitlab-com/people-group/peopleops-eng/connectors-gem
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -214,7 +242,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -222,7 +250,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.2.22
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Library for our shared connectors.
 test_files: []