geet 0.3.13 → 0.3.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 92ec0750a0b9b19fcac605a9dbf4b5a746ac3149eb3685f85ccadae8a7500a94
-  data.tar.gz: 5b56daef325b4a667d2b5aef5b28f5f5f1cdbe2bea6195d6ead8c58173a4580f
+  metadata.gz: 8b37d7ac48684dcd431b13c0e00cb9f999764ae714dec3c8d046102469f7f62f
+  data.tar.gz: 43a045941aff66c212314ae0de2c5c439434c75e89240ce2c59750f5759dcc5a
 SHA512:
-  metadata.gz: de248a32d080a7804b3c2abcb22ad00b6269f2e2d16c30efd5a8b522414e3392e92c563b8798fb59b81d5de6cefeb1b53772ea5cddf13e61d0f94e5d9e25ff27
-  data.tar.gz: 2c6d236e604a29525d73f9a3f356808575b0ddec25bc197436be5badf12063feb6ee2938d8e47e48778814ab912bcb348101c41bd41771ca104044a48ce1fe1a
+  metadata.gz: fe4aa67774fef6b5943ae4f6eb2950ec4d4c2b7f004aa963438e00904d80d01c3a73db8671567c718b6fc9b235a2e9d2002744ec891bce3ff4af1144ffa01d28
+  data.tar.gz: 14c3efa61fa206f3f3f70432c3683d394f9e4e4e09f6cefad757d96d11b88110559f393d22d296acef963d72d7117c668125724da79d8f1f6cbda9e35dfc367a

data/.gitignore

@@ -1,3 +1,4 @@
 /.ruby-version
 /.ruby-gemset
 /Gemfile.lock
+/test_repos

data/.travis.yml

@@ -1,9 +1,17 @@
+dist: bionic
 language: ruby
 rvm:
   - 2.3
   - 2.4
   - 2.5
   - 2.6
+  - 2.7
+  - 3.0
+  - ruby-head
+matrix:
+  fast_finish: true
+  allow_failures:
+    - rvm: ruby-head
 # API tokens are always required, but not used in testing, since no requests are actually made.
 env:
   - GITHUB_API_TOKEN=phony GITLAB_API_TOKEN=phony

data/README.md

@@ -18,15 +18,12 @@ The functionalities currently supported are:
 
 - Github/Gitlab:
   - create label
-  - list issues
-  - list labels
-  - list milestones
-  - list PRs
-  - merge PR
+  - list issues, labels, milestones, MR/PRs
+  - merge MR/PR
+  - open repository
 - Github:
-  - create gist
-  - create issue
-  - create PR
+  - comment PR
+  - create gist, issue, milestone, PR
 
 ## Samples
 

data/bin/geet

@@ -28,39 +28,58 @@ class GeetLauncher
       filename = options.delete(:filename)
       options[:publik] = options.delete(:public) if options.key?(:public)
 
-      Services::CreateGist.new.execute(filename, options)
+      Services::CreateGist.new.execute(filename, **options)
     when ISSUE_CREATE_COMMAND
       summary = options[:summary] || Commandline::Editor.new.edit_content(help: SUMMARY_TEMPLATE)
       title, description = split_summary(summary)
 
       options = default_to_manual_selection(options, :labels, :milestone, :assignees)
 
-      Services::CreateIssue.new(repository).execute(title, description, options)
+      Services::CreateIssue.new(repository).execute(title, description, **options)
     when LABEL_CREATE_COMMAND
       name = options.delete(:name)
 
-      Services::CreateLabel.new(repository).execute(name, options)
+      Services::CreateLabel.new(repository).execute(name, **options)
     when ISSUE_LIST_COMMAND
       options = default_to_manual_selection(options, :assignee)
 
-      Services::ListIssues.new(repository).execute(options)
+      Services::ListIssues.new(repository).execute(**options)
     when LABEL_LIST_COMMAND
       Services::ListLabels.new(repository).execute
+    when MILESTONE_CLOSE_COMMAND
+      # Don't support user selection. This requires extra complexity, specifically, matching by number
+      # while displaying the titles (see AttributesSelectionManager).
+      #
+      options = {numbers: Shared::Selection::MANUAL_LIST_SELECTION_FLAG}
+
+      Services::CloseMilestones.new(repository).execute(**options)
+    when MILESTONE_CREATE_COMMAND
+      title = options.delete(:title)
+
+      Services::CreateMilestone.new(repository).execute(title)
     when MILESTONE_LIST_COMMAND
       Services::ListMilestones.new(repository).execute
+    when PR_COMMENT_COMMAND
+      comment = options.delete(:comment)
+
+      Services::CommentPr.new(repository).execute(comment, **options)
     when PR_CREATE_COMMAND
       summary = options[:summary] || edit_pr_summary(base: options[:base])
       title, description = split_summary(summary)
 
       options = default_to_manual_selection(options, :labels, :milestone, :reviewers)
 
-      Services::CreatePr.new(repository).execute(title, description, options)
+      Services::CreatePr.new(repository).execute(title, description, **options)
     when PR_LIST_COMMAND
       Services::ListPrs.new(repository).execute
     when PR_MERGE_COMMAND
-      Services::MergePr.new(repository).execute(options)
+      Services::MergePr.new(repository).execute(**options)
     when PR_OPEN_COMMAND
-      Services::OpenPr.new(repository).execute(options)
+      Services::OpenPr.new(repository).execute(**options)
+    when REPO_ADD_UPSTREAM_COMMAND
+      Services::AddUpstreamRepo.new(repository).execute
+    when REPO_OPEN_COMMAND
+      Services::OpenRepo.new(repository).execute(**options)
     else
       raise "Internal error - Unrecognized command #{command.inspect}"
     end

data/extra/anonymize_vcr_data

@@ -0,0 +1,58 @@
+#!/bin/bash
+
+set -o errexit
+
+# We match ids with a length of 6 or more chars for simplicity - if we a generic `\d+` match is
+# performed, there are false positives (e.g. PR numbers).
+#
+# Note that since this is an associative array, there is not guaranteed ordering.
+#
+declare -A PATTERNS=(
+  ['Saverio']='Donald'
+  ['saverio']='donald'
+  ['Miroddi']='Duck'
+  ['miroddi']='duck'
+  ['("\w*id"):\d{6,}']='$1:123456'
+  ['\b(\w*id)=\d{6,}']='$1=123456'
+  ['u\/\d{6,}\b']='u\/123456'
+  ['(gravatar.com\/avatar\/)[0-9a-f]{16}']='${1}0123456789abcdef'
+)
+
+TEST_SUITES_LOCATION="$(git rev-parse --show-toplevel)/spec"
+
+if [[ "$1" == "-h" || "$1" == "--help" ]]; then
+  cat <<HELP
+Usage: $(basename "$0") [<files...>|<dirs...>]
+
+Anonymizes the private data in all the files in the project test suites subdirectory ('$TEST_SUITES_LOCATION').
+
+Before applying the changes, all the project files are added to the staging area.
+
+Private data patterns:
+
+HELP
+
+  for key in ${!PATTERNS[@]}; do
+    echo "- $key => ${PATTERNS[$key]}"
+  done
+
+  cat <<'HELP'
+HELP
+
+  exit 0
+fi
+
+if [[ ! -d "$TEST_SUITES_LOCATION" ]]; then
+  echo "The expected TEST_SUITES_LOCATION '$TEST_SUITES_LOCATION' doesn't exist!"
+  exit 1
+fi
+
+git add -A :/
+
+for pattern_from in ${!PATTERNS[@]}; do
+  pattern_to="${PATTERNS[$pattern_from]}"
+
+  grep -lP "$pattern_from" -r "$TEST_SUITES_LOCATION" | xargs -I {} perl -i -pe "s/$pattern_from/$pattern_to/g" "{}"
+done
+
+git difftool --extcmd='vim -d -c "windo set wrap" $5'

data/geet.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |s|
   s.platform    = Gem::Platform::RUBY
   s.required_ruby_version = '>= 2.3.0'
   s.authors     = ['Saverio Miroddi']
-  s.date        = '2019-08-28'
+  s.date        = '2021-05-28'
   s.email       = ['saverio.pub2@gmail.com']
   s.homepage    = 'https://github.com/saveriomiroddi/geet'
   s.summary     = 'Commandline interface for performing SCM host operations, eg. create a PR on GitHub'

data/lib/geet/commandline/commands.rb

@@ -8,11 +8,16 @@ module Geet
       LABEL_CREATE_COMMAND = 'label.create'
       ISSUE_LIST_COMMAND = 'issue.list'
       LABEL_LIST_COMMAND = 'label.list'
+      MILESTONE_CLOSE_COMMAND = 'milestone.close'
+      MILESTONE_CREATE_COMMAND = 'milestone.create'
       MILESTONE_LIST_COMMAND = 'milestone.list'
+      PR_COMMENT_COMMAND = 'pr.comment'
       PR_CREATE_COMMAND = 'pr.create'
       PR_LIST_COMMAND = 'pr.list'
       PR_MERGE_COMMAND = 'pr.merge'
       PR_OPEN_COMMAND = 'pr.open'
+      REPO_ADD_UPSTREAM_COMMAND = 'repo.add_upstream'
+      REPO_OPEN_COMMAND = 'repo.open'
     end
   end
 end

data/lib/geet/commandline/configuration.rb

@@ -45,10 +45,25 @@ module Geet
         ['-u', '--upstream',                                'List on the upstream repository'],
       ].freeze
 
+      MILESTONE_CLOSE_OPTIONS = [
+        long_help: 'Close milestones.'
+      ]
+
+      MILESTONE_CREATE_OPTIONS = [
+        'title',
+        long_help: 'Create a milestone.'
+      ]
+
       MILESTONE_LIST_OPTIONS = [
         ['-u', '--upstream',                                'List on the upstream repository'],
       ].freeze
 
+      PR_COMMENT_OPTIONS = [
+        ['-n', '--no-open-pr',                              "Don't open the PR link in the browser after creation"],
+        'comment',
+        long_help: 'Add a comment to the PR for the current branch.'
+      ]
+
       PR_CREATE_OPTIONS = [
         ['-A', '--automated-mode',                          "Automate the branch operations (see long help)"],
         ['-n', '--no-open-pr',                              "Don't open the PR link in the browser after creation"],
@@ -83,6 +98,15 @@ module Geet
         long_help: 'Open in the browser the PR for the current branch'
       ]
 
+      REPO_ADD_UPSTREAM_OPTIONS = [
+        long_help: 'Add the upstream repository to the current repository (configuration).'
+      ]
+
+      REPO_OPEN_OPTIONS = [
+        ['-u', '--upstream',                                'Open the upstream repository'],
+        long_help: 'Open the current repository in the browser'
+      ]
+
       # Commands decoding table
 
       COMMANDS_DECODING_TABLE = {
@@ -98,14 +122,21 @@ module Geet
           'list' => LABEL_LIST_OPTIONS,
         },
         'milestone' => {
+          'close' => MILESTONE_CLOSE_OPTIONS,
+          'create' => MILESTONE_CREATE_OPTIONS,
           'list' => MILESTONE_LIST_OPTIONS,
         },
         'pr' => {
+          'comment' => PR_COMMENT_OPTIONS,
           'create' => PR_CREATE_OPTIONS,
           'list' => PR_LIST_OPTIONS,
           'merge' => PR_MERGE_OPTIONS,
           'open' => PR_OPEN_OPTIONS,
         },
+        'repo' => {
+          'add_upstream' => REPO_ADD_UPSTREAM_OPTIONS,
+          'open' => REPO_OPEN_OPTIONS,
+        },
       }
 
       # Public interface

data/lib/geet/git/repository.rb

@@ -58,6 +58,10 @@ module Geet
         attempt_provider_call(:Issue, :list, api_interface, assignee: assignee, milestone: milestone)
       end
 
+      def create_milestone(title)
+        attempt_provider_call(:Milestone, :create, title, api_interface)
+      end
+
       def milestone(number)
         attempt_provider_call(:Milestone, :find, number, api_interface)
       end
@@ -66,6 +70,10 @@ module Geet
         attempt_provider_call(:Milestone, :list, api_interface)
       end
 
+      def close_milestone(number)
+        attempt_provider_call(:Milestone, :close, number, api_interface)
+      end
+
       def create_pr(title, description, head, base: nil)
         confirm(LOCAL_ACTION_ON_UPSTREAM_REPOSITORY_MESSAGE) if local_action_on_upstream_repository? && @warnings
         confirm(ACTION_ON_PROTECTED_REPOSITORY_MESSAGE) if action_on_protected_repository? && @warnings
@@ -77,6 +85,12 @@ module Geet
         attempt_provider_call(:PR, :list, api_interface, owner: owner, head: head, milestone: milestone)
       end
 
+      # Returns the RemoteRepository instance.
+      #
+      def remote
+        attempt_provider_call(:RemoteRepository, :find, api_interface)
+      end
+
       # REMOTE FUNCTIONALITIES (ACCOUNT)
 
       def authenticated_user
@@ -115,7 +129,11 @@ module Geet
             raise "The functionality invoked (#{class_name}.#{meth}) is not currently supported!"
           end
 
-          klass.send(meth, *args)
+          # Can't use ruby2_keywords, because the method definitions use named keyword arguments.
+          #
+          kwargs = args.last.is_a?(Hash) ? args.pop : {}
+
+          klass.send(meth, *args, **kwargs)
         else
           raise "The class referenced (#{full_class_name}) is not currently supported!"
         end

data/lib/geet/github/abstract_issue.rb

@@ -61,6 +61,15 @@ module Geet
         @api_interface.send_request(api_path, data: request_data)
       end
 
+      # See https://developer.github.com/v3/issues/comments/#create-a-comment
+      #
+      def comment(comment)
+        api_path = "issues/#{@number}/comments"
+        request_data = { body: comment }
+
+        @api_interface.send_request(api_path, data: request_data)
+      end
+
       # See https://developer.github.com/v3/issues/#edit-an-issue
       #
       def edit(milestone:)

data/lib/geet/github/api_interface.rb

@@ -11,6 +11,8 @@ module Geet
       API_AUTH_USER = '' # We don't need the login, as the API key uniquely identifies the user
       API_BASE_URL = 'https://api.github.com'
 
+      attr_reader :repository_path
+
       # repo_path: optional for operations that don't require a repository, eg. gist creation.
       # upstream:  boolean; makes sense only when :repo_path is set.
       #

data/lib/geet/github/branch.rb

@@ -5,7 +5,7 @@ module Geet
     class Branch
       # See https://developer.github.com/v3/git/refs/#delete-a-reference
       #
-      def self.delete(name, api_interface)
+      def self.delete(name, api_interface, **)
         api_path = "git/refs/heads/#{name}"
 
         api_interface.send_request(api_path, http_method: :delete)

data/lib/geet/github/issue.rb

@@ -6,7 +6,7 @@ module Geet
     autoload :AbstractIssue, File.expand_path('abstract_issue', __dir__)
 
     class Issue < Geet::Github::AbstractIssue
-      def self.create(title, description, api_interface)
+      def self.create(title, description, api_interface, **)
         api_path = 'issues'
         request_data = { title: title, body: description, base: 'master' }
 

data/lib/geet/github/label.rb

@@ -11,7 +11,7 @@ module Geet
       end
 
       # Returns a flat list of names in string form.
-      def self.list(api_interface)
+      def self.list(api_interface, **)
         api_path = 'labels'
         response = api_interface.send_request(api_path, multipage: true)
 
@@ -24,7 +24,7 @@ module Geet
       end
 
       # See https://developer.github.com/v3/issues/labels/#create-a-label
-      def self.create(name, color, api_interface)
+      def self.create(name, color, api_interface, **)
         api_path = 'labels'
         request_data = { name: name, color: color }
 

data/lib/geet/github/milestone.rb

@@ -7,6 +7,8 @@ module Geet
     class Milestone
       attr_reader :number, :title, :due_on
 
+      STATE_CLOSED = 'closed'
+
       class << self
         private
 
@@ -21,9 +23,23 @@ module Geet
         @api_interface = api_interface
       end
 
+      # See https://developer.github.com/v3/issues/milestones/#create-a-milestone
+      def self.create(title, api_interface, **)
+        api_path = 'milestones'
+        request_data = { title: title }
+
+        response = api_interface.send_request(api_path, data: request_data)
+
+        number = response.fetch('number')
+        title = response.fetch('title')
+        due_on = nil
+
+        new(number, title, due_on, api_interface)
+      end
+
       # See https://developer.github.com/v3/issues/milestones/#get-a-single-milestone
       #
-      def self.find(number, api_interface)
+      def self.find(number, api_interface, **)
         api_path = "milestones/#{number}"
 
         response = api_interface.send_request(api_path)
@@ -37,7 +53,7 @@ module Geet
 
       # See https://developer.github.com/v3/issues/milestones/#list-milestones-for-a-repository
       #
-      def self.list(api_interface)
+      def self.list(api_interface, **)
         api_path = 'milestones'
 
         response = api_interface.send_request(api_path, multipage: true)
@@ -50,6 +66,17 @@ module Geet
           new(number, title, due_on, api_interface)
         end
       end
+
+      # See https://docs.github.com/en/free-pro-team@latest/rest/reference/issues#update-a-milestone
+      #
+      # This is a convenience method; the underlying operation is a generic update.
+      #
+      def self.close(number, api_interface, **)
+        api_path = "milestones/#{number}"
+        request_data = { state: STATE_CLOSED }
+
+        api_interface.send_request(api_path, data: request_data)
+      end
     end
   end
 end