ninny 0.1.10 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3d651b7841febd9cbba4fd31c36dcb7882b010ca193ccda73d0260e480ff76e
-  data.tar.gz: 28455776fb99b0e71d069aa1ebf27cd4fa8488723241a9a7686cc51f6ae76169
+  metadata.gz: 1c01f0689e01c71df928ec993bf38c3be68321b1389ef4db7c1ca0c601d0fd02
+  data.tar.gz: 60d84b30dcfe5e8da077e12f59dac45085c94e10d6a060d5d44fa205bbd9196a
 SHA512:
-  metadata.gz: a437f827b80edb78e010deda666d4bec5ea25e5a09e5ecffdc43a44daef569038506174e37326f1da682185b119ad7c3f3e2c5707734a17a5b3f0aedb01aed18
-  data.tar.gz: 9e9d7fea9d69fe4673bcf7acac292bf4ab87a46cd685b3c1f19b02265c8c4ad65e70f3435b26fff9576618592e873fcf20395306df3989efc9bbdbbf6e9e0f4a
+  metadata.gz: 051ef3ceebdc3934e5e2438c25020cc2b7ff8cb1c5428746005c0138658682c8934207fbddb605b03b7f7458ac79d018a56d380bfb8b971075e8c9229642b67a
+  data.tar.gz: 7f3cb4b1f7857bb0380de618016c3d17cd29f9dbdfec9a5d3c1ead4110adaa596a390afae4933b71ef5a9aeca8c9e464878d80c2233149b835bb2660e39d425d

data/.github/workflows/scheduled.yml

@@ -0,0 +1,49 @@
+name: Scheduled
+on:
+  schedule:
+    - cron: "0 4 * * 1" # 4am UTC on Mondays (10pm/11pm CT on Sundays)
+
+jobs:
+  bundler-audit:
+    if:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        fetch-depth: 0
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.7.2
+        bundler: 1.17.3
+        bundler-cache: true
+    - run: |
+        gem install bundler-audit
+        bundler-audit update
+        bundler-audit
+
+        if [ $? -eq 0 ]; then
+          echo "No vulnerabilities found."
+        else
+          echo "Gem vulnerabilities found!"
+          exit 1
+        fi
+  bundle-outdated:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        fetch-depth: 0
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.7.2
+        bundler: 1.17.3
+        bundler-cache: true
+    - run: |
+        bundle outdated --strict
+
+        if [ $? -eq 0 ]; then
+          echo "No outdated gems found."
+        else
+          echo "Outdated gems found!"
+          exit 1
+        fi

data/.rubocop.yml

@@ -11,6 +11,10 @@ Gemspec/RequiredRubyVersion:
 Style/Documentation:
   Enabled: false
 
+Layout/LineLength:
+  Exclude:
+    - spec/**/*
+
 Lint/EmptyClass:
   Enabled: false
 
@@ -20,5 +24,22 @@ Lint/EmptyBlock:
 Lint/MissingSuper:
   Enabled: false
 
+Metrics/BlockLength:
+  Enabled: true
+  Exclude:
+    - spec/**/*
+    - ninny.gemspec
+
+Metrics/MethodLength:
+  Max: 15
+
+Style/AccessModifierDeclarations:
+  EnforcedStyle: inline
+
+Style/BlockDelimiters:
+  Enabled: true
+  Exclude:
+    - spec/**/*
+
 Style/OptionalBooleanParameter:
   Enabled: false

data/README.md

@@ -42,9 +42,15 @@ Then, each developer on the project should set up a file at `~/.ninny.yml` on th
 
 ```bash
 $ ninny setup
-Do you have a new GitLab private token? (Y/n) y # enter 'y'
+Do you have a new GitLab private token? (Y/n) Yes # enter 'y'
 Enter private token: abc123def456ghi789jk # enter your private token
-User config updated
+User config updated!
+```
+
+Or, you can pass the private token in as part of the command:
+```bash
+$ ninny setup --token abc123def456ghi789jk # or '-t abc123def456ghi789jk' for short
+User config updated!
 ```
 
 The private token should be a personal access token for that person's GitLab account (generated [here](https://gitlab.com/-/profile/personal_access_tokens)).
@@ -83,7 +89,7 @@ At any point, `ninny help` will show the help screen.
     * Run `bundle exec guard` to run tests continuously as you develop
 6. Test the gem locally
     * Run `gem build *.gemspec` to build the gem locally
-    * Run `gem install --local ninny-x.x.x.gem` to install the gem locally
+    * Run `gem install --local ninny-X.X.X.gem` to install the gem locally
 7. Make a pull request back to this repository
 
 ### Releasing

data/lib/ninny/cli.rb

@@ -54,6 +54,7 @@ module Ninny
 
     desc 'setup', 'Interactively setup configuration'
     method_option :help, aliases: '-h', type: :boolean, desc: 'Display usage information'
+    method_option :token, aliases: '-t', type: :string, desc: 'The GitLab token to add to the ~/.ninny.yml file'
     def setup(*)
       if options[:help]
         invoke :help, ['setup']

data/lib/ninny/commands/create_dated_branch.rb

@@ -13,11 +13,12 @@ module Ninny
       def execute(output: $stdout)
         create_branch
         delete_old_branches
-        output.puts "#{branch_name} created"
+        output.puts "Branch #{branch_name} successfully created."
       end
 
       # Public: Create the desired branch
       def create_branch
+        prompt.say "Attempting to create branch #{branch_name}."
         Ninny.git.new_branch(branch_name, Ninny.project_config.deploy_branch)
       end
 

data/lib/ninny/commands/pull_request_merge.rb

@@ -35,6 +35,7 @@ module Ninny
 
       # Public: Check out the branch
       def check_out_branch
+        prompt.say "Checking out #{branch_to_merge_into}."
         Ninny.git.check_out(branch_to_merge_into, false)
         Ninny.git.track_current_branch
       rescue Ninny::Git::NoBranchOfType
@@ -44,6 +45,7 @@ module Ninny
 
       # Public: Merge the pull request's branch into the checked-out branch
       def merge_pull_request
+        prompt.say "Merging #{pull_request.branch} to #{branch_to_merge_into}."
         Ninny.git.merge(pull_request.branch)
       end
 

data/lib/ninny/commands/setup.rb

@@ -5,21 +5,29 @@ require_relative '../command'
 module Ninny
   module Commands
     class Setup < Ninny::Command
-      attr_reader :config
+      attr_reader :config, :private_token
 
       def initialize(options)
         @options = options
+        @private_token = options[:token]
         @config = Ninny.user_config
       end
 
       def execute(output: $stdout)
         try_reading_user_config
 
-        prompt_for_gitlab_private_token
+        unless @private_token
+          @private_token = prompt_for_gitlab_private_token
 
-        config.write(force: true)
-        # Command logic goes here ...
-        output.puts "User config #{@result}"
+          unless @private_token
+            output.puts "Please create a private token on GitLab and then rerun 'ninny setup'."
+            return
+          end
+        end
+
+        set_response = config_set_gitlab_private_token(@private_token)
+        write_gitlab_private_token(@private_token, set_response)
+        output.puts "User config #{@result}!"
       end
 
       def try_reading_user_config
@@ -29,6 +37,27 @@ module Ninny
         @result = 'created'
       end
 
+      def config_set_gitlab_private_token(private_token)
+        # TODO: This only works with thor gem < 1. So, we need to make this work when TTY
+        #   releases versions compatible with thor versions >= 1 as well.
+        config.set(:gitlab_private_token, value: private_token)
+        :success
+      rescue ArgumentError
+        puts '  Unable to set new token via TTY... continuing anyway...'
+        :failed
+      end
+
+      def write_gitlab_private_token(private_token, set_response)
+        raise StandardError unless set_response == :success
+
+        # TODO: This only works with thor gem < 1. So, we need to make this work when TTY
+        #   releases versions compatible with thor versions >= 1 as well.
+        config.write(force: true)
+      rescue StandardError
+        puts '  Unable to write config file via TTY... continuing anyway...'
+        File.open("#{ENV['HOME']}/.ninny.yml", 'w') { |file| file.puts "gitlab_private_token: #{private_token}" }
+      end
+
       def prompt_for_gitlab_private_token
         begin
           new_token_text = config.gitlab_private_token ? ' new' : ''
@@ -38,8 +67,7 @@ module Ninny
 
         return unless prompt.yes?("Do you have a#{new_token_text} GitLab private token?")
 
-        private_token = prompt.ask('Enter private token:', required: true)
-        config.set(:gitlab_private_token, value: private_token)
+        prompt.ask('Enter private token:', required: true)
       end
     end
   end

data/lib/ninny/git.rb

@@ -38,7 +38,7 @@ module Ninny
 
     def merge(branch_name)
       if_clean do
-        git.fetch
+        git.fetch('-p')
         command 'merge', ['--no-ff', "origin/#{branch_name}"]
         raise MergeFailed unless clean?
 
@@ -65,8 +65,8 @@ module Ninny
     # branch_name - The name of the branch to check out
     # do_after_pull - Should a pull be done after checkout?
     def check_out(branch, do_after_pull = true)
-      git.fetch
-      branch.checkout
+      git.fetch('-p')
+      git.checkout(branch)
       pull if do_after_pull
       raise CheckoutFailed, "Failed to check out '#{branch}'" unless current_branch.name == branch.name
     end
@@ -84,11 +84,20 @@ module Ninny
     # new_branch_name - The name of the branch to create
     # source_branch_name - The name of the branch to branch from
     def new_branch(new_branch_name, source_branch_name)
-      git.fetch
-      command('branch', ['--no-track', new_branch_name, "origin/#{source_branch_name}"])
-      new_branch = branch(new_branch_name)
-      new_branch.checkout
-      command('push', ['-u', 'origin', new_branch_name])
+      git.fetch('-p')
+      remote_branches = command('branch', ['--remote'])
+
+      if remote_branches.include?("origin/#{new_branch_name}")
+        ask_to_recreate_branch(new_branch_name, source_branch_name)
+      else
+        create_branch(new_branch_name, source_branch_name)
+      end
+    rescue ::Git::GitExecuteError => e
+      if e.message.include?(':fatal: A branch named') && e.message.include?(' already exists')
+        puts "The local branch #{new_branch_name} already exists." \
+          ' Please delete it manually and then run this command again.'
+        exit 1
+      end
     end
 
     # Public: Delete the given branch
@@ -98,13 +107,17 @@ module Ninny
       branch = branch_name.is_a?(::Git::Branch) ? branch_name : git.branch(branch_name)
       git.push('origin', ":#{branch}")
       branch.delete
+    rescue ::Git::GitExecuteError => e
+      if e.message.include?(':error: branch') && e.message.include?(' not found.')
+        puts 'Could not delete local branch, but the remote branch was deleted.'
+      end
     end
 
     # Public: The list of branches on GitHub
     #
     # Returns an Array of Strings containing the branch names
     def remote_branches
-      git.fetch
+      git.fetch('-p')
       git.branches.remote.map { |branch| git.branch(branch.name) }.sort_by(&:name)
     end
 
@@ -159,6 +172,30 @@ module Ninny
       TTY::Prompt.new(options)
     end
 
+    # Private: Ask the user if they wish to delete and recreate a branch
+    #
+    # new_branch_name: the name of the branch in question
+    # source_branch_name: the name of the branch the new branch is supposed to be based off of
+    private def ask_to_recreate_branch(new_branch_name, source_branch_name)
+      if prompt.yes?("The branch #{new_branch_name} already exists. Do you wish to delete and recreate?")
+        delete_branch(new_branch_name)
+        new_branch(new_branch_name, source_branch_name)
+      else
+        exit 1
+      end
+    end
+
+    # Private: Create a branch
+    #
+    # new_branch_name: the name of the branch in question
+    # source_branch_name: the name of the branch the new branch is supposed to be based off of
+    private def create_branch(new_branch_name, source_branch_name)
+      command('branch', ['--no-track', new_branch_name, "origin/#{source_branch_name}"])
+      new_branch = branch(new_branch_name)
+      new_branch.checkout
+      command('push', ['-u', 'origin', new_branch_name])
+    end
+
     # Exceptions
     CheckoutFailed = Class.new(StandardError)
     NotOnBranch = Class.new(StandardError)

data/lib/ninny/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Ninny
-  VERSION = '0.1.10'
+  VERSION = '0.1.15'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ninny
 version: !ruby/object:Gem::Version
-  version: 0.1.10
+  version: 0.1.15
 platform: ruby
 authors:
 - Dispatch Engineers
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-03-01 00:00:00.000000000 Z
+date: 2021-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: git
@@ -286,6 +286,7 @@ extra_rdoc_files: []
 files:
 - ".github/pull_request_template.md"
 - ".github/workflows/main.yml"
+- ".github/workflows/scheduled.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"