git-review 2.0.0.alpha → 2.0.0

data/lib/git-review/server.rb

@@ -0,0 +1,61 @@
+module GitReview
+
+  class Server
+
+    extend Forwardable
+    include ::GitReview::Helpers
+
+    attr_reader :provider
+
+    def self.instance
+      @instance ||= new
+    end
+
+    def initialize
+      init_provider
+    end
+
+    def method_missing(method, *args)
+      if provider.respond_to?(method)
+        provider.send(method, *args)
+      else
+        super
+      end
+    end
+
+    def respond_to?(method)
+      provider.respond_to?(method) || super
+    end
+
+    private
+
+    def init_provider
+      @provider = case
+      when bitbucket_provider?
+        GitReview::Provider::Bitbucket.new
+      when github_provider?
+        GitReview::Provider::Github.new
+      else
+        raise ::GitReview::InvalidGitProviderError
+      end
+    end
+
+    def github_provider?
+      fetch_origin_url =~ %r(github)
+    end
+
+    def bitbucket_provider?
+      fetch_origin_url =~ %r(bitbucket)
+    end
+
+    def fetch_origin_url
+      git_call(call_origin_params)
+    end
+
+    def call_origin_params
+      "config --get remote.origin.url"
+    end
+
+  end
+
+end

data/lib/git-review/settings.rb

@@ -1,48 +1,53 @@
 require 'fileutils'
 require 'yaml'
+require 'hashie'
 
 module GitReview
 
   class Settings
 
-    # acts like a singleton class but it's actually not
-    # use ::GitReview::Settings.instance everywhere except in tests
     def self.instance
       @instance ||= new
     end
 
-    # Read settings from ~/.git_review.yml upon initialization.
-    def initialize
-      @config_file = File.join(Dir.home, '.git_review.yml')
-      @config = YAML.load_file(@config_file) if File.exists?(@config_file)
-      @config ||= {}
-    end
-
-    # Write settings back to file.
     def save!
-      File.open(@config_file, 'w') do |file|
-        file.write(YAML.dump(@config))
-      end
+      File.write(file, dumped)
     end
 
-    # Allow to access config options.
     def method_missing(method, *args)
-      # Determine whether to set or get an attribute.
-      if method.to_s =~ /(.*)=$/
-        @config[$1.to_sym] = args.shift
+      if args.empty?
+        config.send(method)
       else
-        @config[method.to_sym]
+        config.send(method, args.shift)
       end
     end
 
     def respond_to?(method)
-      if method.to_s =~ /(.*)=$/ ||  @config.keys.include?(method.to_sym)
-        true
+      config.respond_to?(method) || super
+    end
+
+    protected
+
+    def config
+      @config ||= Hashie::Mash.new loaded
+    end
+
+    def file
+      @file ||= File.join(Dir.home, '.git_review.yml')
+    end
+
+    def loaded
+      if File.exists? file
+        YAML.load_file(file)
       else
-        super
+        {}
       end
     end
 
+    def dumped
+      YAML.dump config.to_hash
+    end
+
   end
 
 end

data/lib/mixins/accessible.rb

@@ -0,0 +1,35 @@
+module Accessible
+
+  # Setup simple exit criteria for recursion.
+  def accessible?
+    true
+  end
+
+  # Allow to set instance variables on initialization.
+  def initialize(attributes_hash = {})
+    self.update_attributes attributes_hash
+  end
+
+  # Provide access to instance variables like a hash with indifferent access.
+  def [](key)
+    self.instance_variable_get "@#{key}".to_sym
+  end
+
+  # Provide access to instance variables like a hash with indifferent access.
+  def []=(key, value)
+    self.instance_variable_set "@#{key}".to_sym, value
+  end
+
+  # Allow to set all attributes by assigning a hash.
+  def update_attributes(attributes_hash)
+    attributes_hash.each do |key, value|
+      attribute = self[key]
+      if attribute.respond_to? :accessible?
+        attribute.update_attributes(value)
+      else
+        self[key] = value
+      end
+    end
+  end
+
+end

data/lib/mixins/colorizable.rb

@@ -0,0 +1,30 @@
+module Colorizable
+
+  # Takes a color code (= an integer) and formats the Colorizable accordingly.
+  def colorize(color_code)
+    "\e[#{color_code}m#{self}\e[0m"
+  end
+
+  # A couple of presets to keep the code clean.
+
+  def red
+    colorize 31
+  end
+
+  def green
+    colorize 32
+  end
+
+  def yellow
+    colorize 33
+  end
+
+  def blue
+    colorize 34
+  end
+
+  def pink
+    colorize 35
+  end
+
+end

data/lib/mixins/nestable.rb

@@ -0,0 +1,16 @@
+module Nestable
+
+  def nests(mapping)
+    # Setup an accessor for all nested instances.
+    attr_accessor *mapping.keys
+
+    # Create a nested instance automatically on initialize.
+    define_method(:initialize) do |arguments = nil|
+      mapping.each do |attribute, klass|
+        self.instance_variable_set "@#{attribute}".to_sym, klass.new
+      end
+      super arguments if arguments
+    end
+  end
+
+end

data/lib/mixins/string.rb

@@ -0,0 +1,13 @@
+class String
+
+  include Colorizable
+
+  def review_time
+    Time.parse(self).review_time
+  end
+
+  def review_ljust(size)
+    gsub("\n", ' ')[0, size-1].ljust(size)
+  end
+
+end

data/lib/mixins/time.rb

@@ -0,0 +1,11 @@
+class Time
+
+  def review_time
+    strftime('%d-%b-%y')
+  end
+
+  def review_ljust(size)
+    to_s.review_ljust(size)
+  end
+
+end

data/lib/models/commit.rb

@@ -0,0 +1,14 @@
+class Commit
+
+  include Accessible
+  extend Nestable
+
+  nests user: User,
+        repository: Repository
+
+  attr_accessor :sha,
+                :ref,
+                :label,
+                :repo
+
+end

data/lib/models/repository.rb

@@ -0,0 +1,8 @@
+class Repository
+
+  include Accessible
+
+  attr_accessor :owner,
+                :name
+
+end

data/lib/models/request.rb

@@ -0,0 +1,18 @@
+class Request
+
+  include Accessible
+  extend Nestable
+
+  nests head: Commit
+
+  attr_accessor :number,
+                :title,
+                :body,
+                :state,
+                :html_url,
+                :patch_url,
+                :updated_at,
+                :comments,
+                :review_comments
+
+end

data/lib/models/user.rb

@@ -0,0 +1,7 @@
+class User
+
+  include Accessible
+
+  attr_accessor :login
+
+end

metadata

@@ -1,112 +1,127 @@
 --- !ruby/object:Gem::Specification
 name: git-review
 version: !ruby/object:Gem::Version
-  version: 2.0.0.alpha
-  prerelease: 6
+  version: 2.0.0
 platform: ruby
 authors:
 - Dominik Bamberger
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-04 00:00:00.000000000 Z
+date: 2013-10-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: launchy
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: octokit
+  name: yajl-ruby
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: yajl-ruby
+  name: hashie
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: gli
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.8.0
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: octokit
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
       - !ruby/object:Gem::Version
         version: 2.13.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
         version: 2.13.0
 - !ruby/object:Gem::Dependency
-  name: hashie
+  name: guard
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.3
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.1.0
 description: Manage review workflow for projects hosted on GitHub (using pull requests).
 email: bamberger.dominik@gmail.com
 executables:
@@ -118,34 +133,44 @@ files:
 - lib/git-review.rb
 - lib/git-review/settings.rb
 - lib/git-review/commands.rb
+- lib/git-review/helpers.rb
+- lib/git-review/server.rb
 - lib/git-review/errors.rb
-- lib/git-review/github.rb
-- lib/git-review/internals.rb
 - lib/git-review/local.rb
+- lib/git-review/provider/github.rb
+- lib/git-review/provider/base.rb
+- lib/git-review/provider/bitbucket.rb
+- lib/mixins/colorizable.rb
+- lib/mixins/string.rb
+- lib/mixins/accessible.rb
+- lib/mixins/time.rb
+- lib/mixins/nestable.rb
+- lib/models/user.rb
+- lib/models/commit.rb
+- lib/models/repository.rb
+- lib/models/request.rb
 - bin/git-review
 homepage: http://github.com/b4mboo/git-review
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>'
+  - - '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
-summary: facilitates GitHub code reviews
+specification_version: 4
+summary: Facilitates GitHub code reviews
 test_files: []
-has_rdoc: