reviewlette 0.0.6

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 53593949368f61246646984e517e18c7e478497c
+  data.tar.gz: 2c1af0b6ac234d1a06ea28f1906323857a398073
+SHA512:
+  metadata.gz: 1e3e7d9571d7869b2f49e34a7d1be93269e8f8c6f300f00f02bb757e3036aeab4248dfddb8a9479735aac970ebbad56765b6defbfd4bcfc234ac3a94476bf60f
+  data.tar.gz: f4fbe33ad2be3922ae2e9de8d494209f2defbecb0cd19f49affed7d72b4c88cd7d2bbd3b7e942920644f33f2349f74ed83f1995884441f222c0fe3ff92d5f2dc

data/.gitignore

@@ -0,0 +1,27 @@
+*.gem
+*.db
+*.log
+*.html
+database
+*.yml~
+.idea
+/.idea
+.idea*
+/.idea/*
+*.rbc
+*.yml
+.bundle
+coverage
+.swp
+InstalledFiles
+lib/bundler/man
+pkg
+rdoc
+spec/reports
+test/tmp
+test/version_tmp
+tmp
+.yardoc
+_yardoc
+rdoc/
+doc/

data/.rspec

@@ -0,0 +1 @@
+--colour --format documentation 

data/Gemfile

@@ -0,0 +1,17 @@
+source 'http://rubygems.org'
+
+gem 'json'
+gem 'haml'
+gem 'octokit', '~> 3.0'
+gem 'ruby-trello'
+gem 'rdoc'
+gem 'rspec'
+gem 'webmock'
+gem 'rake'
+gem 'ap'
+gem 'byebug'
+gem 'simplecov'
+gem 'prophet'
+gem 'coveralls'
+gem 'sequel'
+gem 'sqlite3'

data/Gemfile.lock

@@ -0,0 +1,114 @@
+GEM
+  remote: http://rubygems.org/
+  specs:
+    activemodel (4.1.1)
+      activesupport (= 4.1.1)
+      builder (~> 3.1)
+    activesupport (4.1.1)
+      i18n (~> 0.6, >= 0.6.9)
+      json (~> 1.7, >= 1.7.7)
+      minitest (~> 5.1)
+      thread_safe (~> 0.1)
+      tzinfo (~> 1.1)
+    addressable (2.3.6)
+    ap (0.1.1)
+      httparty (>= 0.7.7)
+    builder (3.2.2)
+    byebug (3.1.2)
+      columnize (~> 0.8)
+    columnize (0.8.9)
+    coveralls (0.7.0)
+      multi_json (~> 1.3)
+      rest-client
+      simplecov (>= 0.7)
+      term-ansicolor
+      thor
+    crack (0.4.2)
+      safe_yaml (~> 1.0.0)
+    diff-lcs (1.2.5)
+    docile (1.1.3)
+    faraday (0.9.0)
+      multipart-post (>= 1.2, < 3)
+    haml (4.0.5)
+      tilt
+    httparty (0.13.1)
+      json (~> 1.8)
+      multi_xml (>= 0.5.2)
+    i18n (0.6.9)
+    json (1.8.1)
+    mime-types (2.3)
+    minitest (5.3.4)
+    multi_json (1.10.0)
+    multi_xml (0.5.5)
+    multipart-post (2.0.0)
+    oauth (0.4.7)
+    octokit (3.1.0)
+      sawyer (~> 0.5.3)
+    prophet (1.5.5)
+      octokit
+      rspec
+    rake (10.3.2)
+    rdoc (4.1.1)
+      json (~> 1.4)
+    rest-client (1.6.7)
+      mime-types (>= 1.16)
+    rspec (3.0.0)
+      rspec-core (~> 3.0.0)
+      rspec-expectations (~> 3.0.0)
+      rspec-mocks (~> 3.0.0)
+    rspec-core (3.0.0)
+      rspec-support (~> 3.0.0)
+    rspec-expectations (3.0.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.0.0)
+    rspec-mocks (3.0.0)
+      rspec-support (~> 3.0.0)
+    rspec-support (3.0.0)
+    ruby-trello (1.1.1)
+      activemodel (>= 3.2.0)
+      addressable (~> 2.3)
+      json
+      oauth (~> 0.4.5)
+      rest-client (~> 1.6.7)
+    safe_yaml (1.0.3)
+    sawyer (0.5.4)
+      addressable (~> 2.3.5)
+      faraday (~> 0.8, < 0.10)
+    sequel (4.13.0)
+    simplecov (0.8.2)
+      docile (~> 1.1.0)
+      multi_json
+      simplecov-html (~> 0.8.0)
+    simplecov-html (0.8.0)
+    sqlite3 (1.3.9)
+    term-ansicolor (1.3.0)
+      tins (~> 1.0)
+    thor (0.19.1)
+    thread_safe (0.3.4)
+    tilt (2.0.1)
+    tins (1.1.0)
+    tzinfo (1.2.1)
+      thread_safe (~> 0.1)
+    webmock (1.18.0)
+      addressable (>= 2.3.6)
+      crack (>= 0.3.2)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  ap
+  byebug
+  coveralls
+  haml
+  json
+  octokit (~> 3.0)
+  prophet
+  rake
+  rdoc
+  rspec
+  ruby-trello
+  sequel
+  simplecov
+  sqlite3
+  webmock

data/Guardfile

@@ -0,0 +1,24 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+guard :rspec do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
+  watch('spec/spec_helper.rb')  { "spec" }
+
+  # Rails example
+  watch(%r{^app/(.+)\.rb$})                           { |m| "spec/#{m[1]}_spec.rb" }
+  watch(%r{^app/(.*)(\.erb|\.haml|\.slim)$})          { |m| "spec/#{m[1]}#{m[2]}_spec.rb" }
+  watch(%r{^app/controllers/(.+)_(controller)\.rb$})  { |m| ["spec/routing/#{m[1]}_routing_spec.rb", "spec/#{m[2]}s/#{m[1]}_#{m[2]}_spec.rb", "spec/acceptance/#{m[1]}_spec.rb"] }
+  watch(%r{^spec/support/(.+)\.rb$})                  { "spec" }
+  watch('config/routes.rb')                           { "spec/routing" }
+  watch('app/controllers/application_controller.rb')  { "spec/controllers" }
+
+  # Capybara features specs
+  watch(%r{^app/views/(.+)/.*\.(erb|haml|slim)$})     { |m| "spec/features/#{m[1]}_spec.rb" }
+
+  # Turnip features and steps
+  watch(%r{^spec/acceptance/(.+)\.feature$})
+  watch(%r{^spec/acceptance/steps/(.+)_steps\.rb$})   { |m| Dir[File.join("**/#{m[1]}.feature")][0] || 'spec/acceptance' }
+end
+

data/LICENSE

@@ -0,0 +1,21 @@
+The MIT License (MIT)
+
+Copyright (c) 2014 jschmid1
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

data/README.md

@@ -0,0 +1,95 @@
+Reviewlette
+===========
+
+[![Coverage Status](https://img.shields.io/coveralls/jschmid1/reviewlette.svg)](https://coveralls.io/r/jschmid1/reviewlette)
+[![Code Climate](https://codeclimate.com/github/jschmid1/reviewlette.png)](https://codeclimate.com/github/jschmid1/reviewlette)
+
+Tool to automatically assign a "Reviewer" to a GitHub Issue and to the attached Trello Card.
+
+
+What it does:
+
+- Finds unassigned issues on GitHub.
+- Assignes a member of your team.
+- Locates the right Card on Trello.
+- Checks if the assignee is on vacation(using tel).
+- Adds the assigned member to the Card.
+- If the Issue/PullRequest is closed or merged move it in the right column.
+- Prints graphs using Morris.js to display statistics(autorefresh every 5 seconds).
+
+
+![alt tag](http://h.dropcanvas.com/72fj0/graph.jpg)
+
+
+## Installation
+
+```
+git clone git@github.com:jschmid1/reviewlette.git
+cd reviewlette
+bundle
+cd bin
+./reviewlette
+```
+
+or
+
+```
+gem install reviewlette
+```
+
+
+## Setup
+
+#### Name your pullrequest like so:
+#### Review_#23_name_of_review_42  <= trello card number
+
+Fill `config/.trello.yml` with your **consumerkey**, **consumersecret**, **oauthtoken** and **board_id**
+
+```yml
+-comsumerkey: theconsumerkey11
+-consumersecret: theconsumersecret11
+-oauthtoken: theoauthtoken11
+```
+
+[Which can be generated here](https://trello.com/1/appKey/generate)
+
+Fill `config/.github.yml` with your **token** and **repo**
+
+```yml
+-token: thetokenfromgithub
+-repo: ['my/repo', 'my/otherrepo']
+```
+
+[Which can be generated here](https://github.com/settings/applications/new)
+
+
+Edit the `reviewlette.db` scaffold in the main directory to your needs.
+
+Structure:
+
+| primary_key   | first_name     | last_name     | Reviews_count  | created_at    | github_name   | trello_name   | vacation_status | tel_name      |
+|:-------------:|:-------------: |:-------------:|:-------------: |:-------------:|:-------------:|:-------------:|:-------------:  |:-------------:|
+| Integer(PK)   | Text           | Text          | Integer        | Text          |  Text         |Text           |          Numeric|           Text|
+| 1             | Joshua         | Schmid        | 30             | 2014-01-01    |  jschmid1     |jschmid1       |          false  |        jschmid|
+
+
+
+
+You can either use a GUI like [Sqlite database browser](http://sqlitebrowser.org/) or the sqlite commandline interface
+
+e.g.
+
+```ruby
+insert into reviewer values('John','Smith', '0', '', 'github_name', 'trello_name', 'false', 'tel_name');
+```
+
+---
+
+[Using Octokit as a GitHub api wrapper](https://github.com/octokit/octokit.rb)
+
+[Using Sequel as Database Module](https://github.com/jeremyevans/sequel)
+
+[Using ruby-trello as a Trello api wrapper](https://github.com/jeremytregunna/ruby-trello)
+
+
+

data/Rakefile

@@ -0,0 +1,5 @@
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:spec)
+
+task :default => :spec

data/bin/reviewlette

@@ -0,0 +1,7 @@
+#!/usr/bin/ruby
+
+$LOAD_PATH.unshift File.join(File.dirname(__FILE__), '..', 'lib')
+require 'reviewlette'
+
+Reviewlette.spin
+

data/lib/reviewlette.rb

@@ -0,0 +1,202 @@
+require 'reviewlette/trello_connection'
+require 'reviewlette/github_connection'
+require 'reviewlette/database'
+require 'reviewlette/graph_gen'
+require 'reviewlette/vacations'
+require 'yaml'
+require 'octokit'
+require 'trello'
+
+class Trello::Card
+
+  def assignees
+    @trello_connection = Reviewlette::TrelloConnection.new
+    member_ids.map{|id| @trello_connection.find_member_by_id(id)}
+  end
+end
+
+module Reviewlette
+
+  attr_accessor :trello_connection, :github_connection, :repo, :board, :db
+
+  TRELLO_CONFIG1 = YAML.load_file("#{File.dirname(__FILE__)}/../config/.trello.yml")
+
+  class << self
+
+    # Execute logic.
+    def spin
+      setup
+      get_available_repos.each do |repo|
+        @repo = repo
+        get_unassigned_github_issues.each do |issue|
+          @number = issue[:number]
+          @title = issue[:title]
+          @body = issue[:body]
+          update_vacations
+          
+          if find_card(@title.to_s) and find_id
+            if set_reviewer
+              transform_name
+              add_reviewer_on_github
+              comment_on_github
+              # add_to_trello_card
+              comment_on_trello
+              move_to_list
+              @db.add_pr_to_db(@title, @reviewer.username)
+              @reviewer = nil
+              Reviewlette::Graphgenerator.new.write_to_graphs('graph.html',Reviewlette::Graphgenerator.new.model_graphs(Reviewlette::Database.new.conscruct_line_data.to_json, Reviewlette::Database.new.conscruct_graph_struct.to_json, 'Donut'))
+            else
+              comment_on_error
+            end
+          end
+          puts 'No new issues.' unless @issues.present?
+        end
+      end
+    end
+
+    def get_available_repos
+      @repos = Reviewlette::GithubConnection::GITHUB_CONFIG['repo']
+    end
+
+    # Finds card based on title of Github Issue.
+    # Or by branch name if title does not include the trello number.
+    # Happens if the pullrequest consists of only one commit.
+    def find_card(title)
+      begin
+        match_pr_id_with_issue_id
+        @id = title.split('_').last.to_i
+        @id = fetch_branch if @id == 0 && !(@pullreq_ids.values.index(@number)).nil?
+        raise NoTrelloCardException, "Could not find a Trello card. Found #{title.split('_').last} instead, maybe the naming of your pullrequest is wrong? And/or you dont have a branch?" if @id == 0
+        true
+      rescue NoTrelloCardException => e
+        @logger.error e.message
+        puts (e.message)
+        false
+      end
+    end
+
+    def update_vacations
+      @db.get_users_tel_entries.each do |name|
+        @vacations = Reviewlette::Vacations.find_vacations(name) #tel_name
+        evaluate_vacations(name)
+      end
+    end
+
+    def evaluate_vacations(reviewer)
+      parse_vacations.each do |check|
+        check[1] = check[0] unless check[1] # Rewrite if statement with catch to prevent this error?
+        if (check[0]..check[1]).cover?(Date.today)
+          @db.set_vacation_flag(reviewer, 'true')
+          break
+        else
+          @db.set_vacation_flag(reviewer, 'false') # hopefully not to_bool?
+        end
+      end
+    end
+
+    def parse_vacations
+      split = @vacations.map { |x| x.split(' - ') }
+      split.map { |x| x.map { |b| Date.parse(b) } }
+    end
+
+    # gets the branchname from github pullrequest
+    def fetch_branch
+      pr_id = @pullreq_ids.values.index(@number)
+      branch_name = @github_connection.get_branch_name(pr_id, @repo)
+      branch_name.split('_').last.to_i
+    end
+
+    # Matches Pull Request IDs with the respective Order of PullRequests
+    # to call them and get the branch name.
+    def match_pr_id_with_issue_id
+      @pullreq_ids = {}
+      @counter = 0
+      @github_connection.list_pulls(@repo).each do |x|
+        @pullreq_ids[@counter] = x.number
+        @counter +=1
+      end
+    end
+
+    # TODO: Generic Error message.
+    def comment_on_error
+      @trello_connection.comment_on_card("Skipped Issue #{@number} because everyone on the team is assigned to the card", @card)
+    end
+
+    # Gets [Array(String, String)] all GitHub Issues that are not assigned to someone.
+    def get_unassigned_github_issues
+      @issues = @github_connection.list_issues(@repo).select{|issue| !issue[:assignee]}
+    end
+
+    # Sets instance variables.
+    def setup
+      @logger = Logger.new('review.log')
+      @db = Reviewlette::Database.new
+      @github_connection = Reviewlette::GithubConnection.new
+      @trello_connection = Reviewlette::TrelloConnection.new
+      @board = Trello::Board.find(TRELLO_CONFIG1['board_id'])
+    end
+
+    # Finds a sets card.
+    def find_id
+      if @id != 0
+        @card = @trello_connection.find_card_by_id(@id)
+        true
+      else
+        @logger.warn("Id not found, skipping Issue #{@title} with number #{@number}")
+        false
+      end
+    end
+
+    # Selects and sets reviewer.
+    def set_reviewer
+      begin
+        while !(@reviewer)
+          @reviewer = @trello_connection.determine_reviewer(@card) if @card
+        end
+        @trelloname = @reviewer.username
+        puts "Selected #{@reviewer.username}"
+        return true
+      rescue AlreadyAssignedException => e
+        @logger.warn("Skipped Issue #{@card.short_id} because #{e.message}")
+        puts ("Skipped Issue #{@card.short_id} because #{e.message}")
+        return false
+      end
+    end
+
+    # Get Trelloname from configfile.
+    def transform_name
+      @githubname = @db.find_gh_name_by_trello_name(@trelloname)
+    end
+
+    # Adds Assignee on GitHub.
+    def add_reviewer_on_github
+      @github_connection.add_assignee(@repo, @number, @title, @body, @githubname) if @number && @githubname
+    end
+
+    # Comments on GitHub.
+    def comment_on_github
+      @github_connection.comment_on_issue(@repo, @number, @githubname, @card.url) if @number && @githubname
+    end
+    # Adds Reviewer on Trello Card.
+    def add_to_trello_card
+      @trello_connection.add_reviewer_to_card(@reviewer, @card)
+    end
+
+    # Comments on Trello Card.
+    def comment_on_trello
+      @full_comment = '@' + @trelloname + ' will review ' + 'https://github.com/'+ @repo+'/issues/'+@number.to_s
+      @trello_connection.comment_on_card(@full_comment, @card) if @full_comment
+    end
+
+    # TODO: More generic 'Done' and 'in Review' are not present everywhere
+    def move_to_list
+      if @github_connection.pull_merged?(@repo, @id)
+        @column = @trello_connection.find_column('Done')
+        @trello_connection.move_card_to_list(@card, @column)
+      else
+        @column = @trello_connection.find_column('In review')
+        @trello_connection.move_card_to_list(@card, @column)
+      end
+    end
+  end
+end