tutter-sppuppet 0.0.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: fb3374056a2601ef388af3b3119d6fc236e00030
+  data.tar.gz: acc0ca1594ca46c7ef6bb4e701311f1d41a4d616
+SHA512:
+  metadata.gz: df26292e971f810911aec4f64885f177d59057374ec707afae0f299c272f2e57fbda41d2f0b3d08b7a252d92681fcd0062534ec898562f302aa2b733d53d88e3
+  data.tar.gz: ed4f4058f138b80b8fb86d9946450ad21fd4b03b65f67631858271043c7d7b8931308fe38d4352dfd59bc3b4070b0bcbd2bb752b07381b0bbe2cff5e6c073939

data/.gemspec

@@ -0,0 +1,16 @@
+# -*- encoding: utf-8 -*-
+Gem::Specification.new do |s|
+  s.name        = 'tutter-sppuppet'
+  s.version     = '0.0.1'
+  s.author      = 'Johan Haals'
+  s.email       = ['johan.haals@gmail.com']
+  s.homepage    = 'https://github.com/jhaals/tutter-sppuppet'
+  s.summary     = 'Github code review without collaborator access'
+  s.description = 'This tutter action let non collaborators review and merge code without having more then read access to the project'
+  s.license     = 'Apache 2.0'
+
+  s.files         = `git ls-files`.split("\n")
+  s.require_paths = ['lib']
+
+  s.required_ruby_version = '>= 1.8.7'
+end

data/README.md

@@ -0,0 +1,23 @@
+# tutter-sppuppet
+
+This action let non collaborators review
+and merge code without having more then read access to the project.
+
+1. A pull request get submitted
+2. Someone thinks it looks good and adds a _+1_ comment
+3. Another person comment _+1_
+4. The pull request can be merged by commenting _!merge_ when it has the
+desired amount of +1's(configurable)
+
+A pull request will be blocked if it has a _-1_ comment
+
+
+### tutter.yaml sppuppet specific settings
+
+    action: 'sppuppet'
+    action_settings:
+    plus_ones_required: 3
+
+### TODO
+* whitelist
+* blacklist

data/lib/tutter/action/sppuppet.rb

@@ -0,0 +1,52 @@
+class Sppuppet
+
+  def initialize(settings, client, project, data)
+    @settings = settings
+    @client = client
+    @project = project
+    @data = data
+  end
+
+  def run
+    pull_request_id = @data['issue']['number']
+    pr = @client.pull_request @project, pull_request_id
+    plus_one = {}
+    merge = false
+
+    if pr.mergeable_state != 'clean'
+      puts "merge state for #{@project} #{pull_request_id} is not clean. Current state: #{pr.mergeable_state}"
+      return false
+    end
+
+    # Don't care about code we can't merge
+    return false unless pr.mergeable
+
+    comments = @client.issue_comments(@project, pull_request_id)
+
+    # Check each comment for +1 and merge comments
+    comments.each do |i|
+
+      if /^(\+1|:\+1)/.match i.body
+        # pull request submitter cant +1
+
+        unless pr.user.login == i.attrs[:user].attrs[:login]
+          plus_one[i.attrs[:user].attrs[:login]] = 1
+        end
+      end
+
+      # TODO it should calculate the +1's - the -1's
+      # Never merge if someone says -1
+      if /^(\-1|:\-1:)/.match i.body
+        puts "#{@project} #{pull_request_id} has a -1. I will not take the blame"
+        return false
+      end
+    end
+
+    merge = true if comments.last.body == '!merge'
+
+    if plus_one.count >= @settings['plus_ones_required'] and merge
+      puts "merging #{pull_request_id} #{@project}"
+      @client.merge_pull_request(@project, pull_request_id, 'SHIPPING!!')
+    end
+  end
+end

metadata

@@ -0,0 +1,48 @@
+--- !ruby/object:Gem::Specification
+name: tutter-sppuppet
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+platform: ruby
+authors:
+- Johan Haals
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2014-02-03 00:00:00.000000000 Z
+dependencies: []
+description: This tutter action let non collaborators review and merge code without
+  having more then read access to the project
+email:
+- johan.haals@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- .gemspec
+- README.md
+- lib/tutter/action/sppuppet.rb
+homepage: https://github.com/jhaals/tutter-sppuppet
+licenses:
+- Apache 2.0
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: 1.8.7
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.0.3
+signing_key: 
+specification_version: 4
+summary: Github code review without collaborator access
+test_files: []