pra 1.7.2 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c086ca34fcf440f0e516fc83da1a225d20a6261c
-  data.tar.gz: d33e01807fc0d728f458cb038a36b6afec9334f4
+  metadata.gz: e4c0d01126812263dbcf4816f1dcd3383e99d702
+  data.tar.gz: 99804a97e1c2dc20cc06510e52a1c4abd9fae6ad
 SHA512:
-  metadata.gz: e9e6a805ec6a32b396b12e8ef620d0ab6c81281361731b85f26804bb0b93c616a6a72c51d312d23fa7ccc4c538b2137234eb5f286cda3445f96e449effcd3977
-  data.tar.gz: 3245aaf01131d7909c8ef3346cad47029e9c6abd1d198c837e5e369dfd715c68ecdf524bf5e4d82921bb376424af3e825935603c70b0044efaa632bd72e11fa3
+  metadata.gz: 1b2e1d2f91e56a3954cc2a8236e0ddf481b19da67f108173bb3a52a4e545639040b6168b1901896d6368dd0223a523540a7e4232b0c0f716578133d5da55750f
+  data.tar.gz: 80296a40d2becad79596672025d243d58b300cfbbc58419b6d3b246f00fa3c75b68b1d43c37ababc8e4d10c53be99f173bdf195f062ea3e25782932d11f556af

data/ChangeLog.md

@@ -6,6 +6,16 @@ versions as well as provide a rough history.
 
 #### Next Release
 
+#### v2.0.0
+* Added ability to refresh pull requests
+* Added pagination when there are more pull requests than can be displayed
+* Added support for github organizations
+* Added last refresh time to UI
+* Redesgined pull request render process to make format changes easier
+* Added github labels
+* Changed github query method to reduce API calls and support organizations
+* Added github rate limiting protection
+
 #### v1.7.2
 
 * Fix bug #2, left over PRs not clearing

data/README.md

@@ -13,13 +13,21 @@ You can easily install `pra` with the following command:
 
     $ gem install pra
 
+## Migrating to 2.0
+
+The configuration location has moved from `~/.pra.json` to
+`~/.pra/config.json`. The current version of pra reads from both locations but
+it is preferred to move your configuration file to the `.pra` folder. This
+folder will automatically be created, if it does not exist, for logging
+purposes when pra starts.
+
 ## Configuration
 
 `pra` requires one configuration, `~/.pra.json`, to exist in your home
 directory. The following is an example config that can be used as a starter.
 *Note:* You will need to replace **your.username**, **your.password**,
-**your.stash.server**, and the **repositories** sections of each of the pull
-sources.
+**your.stash.server**, and the **repositories** and **organizations** sections
+of each of the pull sources.
 
     {
       "pull_sources": [
@@ -47,6 +55,9 @@ sources.
             "repositories": [
               { "owner": "reachlocal", "repository": "snapdragon" },
               { "owner": "brewster", "repository": "cequel" }
+            ],
+            "organizations": [
+              { "name": "codebreakdown", "exclude": ["snapdragon"]}
             ]
           }
         }
@@ -54,14 +65,17 @@ sources.
       "assignee_blacklist": [
         "IPT-Capture",
         "IPT-Core Services"
-      ]
+      ],
+      "refresh_interval": 300,
+      "log_level": "info"
     }
 
-I suggest copying and pasting the above starter file into your `~/.pra.json`
+I suggest copying and pasting the above starter file into your `~/.pra/config.json`
 file to get you started. Then simply replace the appropriate fields and the
-**repositories** sections for all the pull sources with the repository
+**repositories** and **organizations** sections for all the pull sources with the repository
 information for the repositories you want to watch for open pull requests.
 
+
 #### Stash User & Project Repositories
 
 You may have noticed that the above example shows two repository objects in its
@@ -81,6 +95,13 @@ scenarios where the repository is housed under a project.
 Reduces noise to more easily determine which pull requests are unassigned. Names
 added will not appear in the assignee column.
 
+#### Github Organizations
+
+Instead of listing each repository, an organization can be provided and all
+pull requests open for projects in that organization will be listed. To
+exclude any unwanted projects, add the repository name to the `exclude` array
+for that organization.
+
 ### GitHub Authentication
 
 #### Multi-Factor Authentication
@@ -119,9 +140,10 @@ running the following command:
 
     pra
 
-Once it launches, it will use the information provided in the `~/.pra.json`
-configuration file to fetch all the open pull requests and display them. Once,
-the pull requests are displayed you can perorm any of the following actions.
+Once it launches, it will use the information provided in the
+`~/.pra/config.json` configuration file to fetch all the open pull requests
+and display them. Once, the pull requests are displayed you can perform any of
+the following actions.
 
 ### Move Selection Up
 
@@ -136,6 +158,15 @@ To move the selection down simply press either the `j` or `down arrow` key.
 If you would like to open the currently selected pull request in your default
 browser you can press either the `o` or `enter` key.
 
+### Refresh
+
+To force a refresh press the `r` key.
+
+### Move Between Pages
+
+To move between pages press `n` to go to the next page, and `p` to go to the
+previous page.
+
 ### Quit
 
 If you decide you have had enough and  want to exit `pra` press the `q` key.

data/lib/pra/app.rb

@@ -2,7 +2,7 @@ require 'thread'
 
 require 'pra/window_system_factory'
 require 'pra/pull_request_service'
-require 'pra/error_log'
+require 'pra/log'
 
 Thread.abort_on_exception=true
 
@@ -22,6 +22,15 @@ module Pra
     end
 
     def fetch_and_refresh_pull_requests
+      if @window_system.force_refresh || Time.now - @window_system.last_updated > Pra.config.refresh_interval
+        refresh_pull_requests
+      end
+
+      Kernel.sleep(0.1)
+    end
+
+    def refresh_pull_requests
+      @window_system.force_refresh = false
       @window_system.fetching_pull_requests
       new_pull_requests = []
 
@@ -31,14 +40,12 @@ module Pra
         end
 
         fetch.on_error do |error|
-          Pra::ErrorLog.log(error)
+          Pra::Log.error(error)
           @window_system.fetch_failed
         end
       end
 
       @window_system.refresh_pull_requests(new_pull_requests)
-      
-      Kernel.sleep(5 * 60)
     end
 
     def pull_request_fetcher_thread

data/lib/pra/config.rb

@@ -1,9 +1,13 @@
 require 'json'
+require 'fileutils'
 
 module Pra
   class Config
     def initialize(initial_config = {})
       @initial_config = initial_config
+      if @initial_config["log_level"]
+        Pra::Log.level(@initial_config["log_level"])
+      end
     end
 
     def self.load_config
@@ -22,15 +26,18 @@ module Pra
     end
 
     def self.config_path
-      return File.join(self.users_home_directory, '.pra.json')
-    end
-
-    def self.error_log_path
-      return File.join(self.users_home_directory, '.pra.errors.log')
+      if File.exists?(File.join(self.users_home_directory, '.pra', 'config.json'))
+        return File.join(self.users_home_directory, '.pra', 'config.json')
+      else
+        return File.join(self.users_home_directory, '.pra.json')
+      end
     end
 
     def self.log_path
-      return File.join(self.users_home_directory, '.pra.log')
+      unless Dir.exists?(File.join(self.users_home_directory, '.pra', 'logs'))
+        FileUtils.mkdir_p(File.join(self.users_home_directory, '.pra', 'logs'))
+      end
+      return File.join(self.users_home_directory, '.pra', 'logs', '.pra.log')
     end
 
     def self.users_home_directory
@@ -48,5 +55,9 @@ module Pra
     def assignee_blacklist
       Array(@initial_config["assignee_blacklist"])
     end
+    
+    def refresh_interval
+      @initial_config["refresh_interval"] || 60*5
+    end
   end
 end

data/lib/pra/curses_pull_request_presenter.rb

@@ -1,4 +1,5 @@
 require 'pra/config'
+require 'time-lord'
 
 module Pra
   class CursesPullRequestPresenter
@@ -7,37 +8,56 @@ module Pra
     end
 
     def repository
-      force_length(@pull_request.repository, 15)
+      @pull_request.repository
     end
 
     def title
-      force_length(@pull_request.title, 20)
+      @pull_request.title
     end
 
     def from_reference
-      force_length(@pull_request.from_reference, 20)
+      @pull_request.from_reference
     end
 
     def to_reference
-      force_length(@pull_request.to_reference, 20)
+      @pull_request.to_reference
     end
 
     def author
-      force_length(@pull_request.author, 20)
+      @pull_request.author
     end
 
     def assignee
-      return force_length('', 20) if @pull_request.assignee.nil? || blacklisted?(@pull_request.assignee)
-      force_length(@pull_request.assignee, 20)
+      if @pull_request.assignee.nil? || blacklisted?(@pull_request.assignee)
+        return ""
+      else
+        @pull_request.assignee
+      end
     end
 
     def service_id
-      force_length(@pull_request.service_id, 10)
+      @pull_request.service_id
+    end
+
+    def labels
+      @pull_request.labels
+    end
+
+    def updated_at
+      @pull_request.updated_at.to_time.ago.to_words
     end
 
     def assignee_blacklist
-      config = Pra::Config.load_config
-      config.assignee_blacklist
+      Pra.config.assignee_blacklist
+    end
+
+    def present(columns)
+      row = ""
+      columns.each do |column|
+        row << force_length(send(column[:name]), column[:size])
+        row << (" " * column[:padding])
+      end
+      row
     end
 
     private

data/lib/pra/curses_window_system.rb

@@ -10,10 +10,37 @@ module Pra
     ENTER_KEY = 10
 
     def initialize
-      @selected_pull_request_index = 0
+      @selected_pull_request_page_index = 0
+      @current_page = 1
       @current_pull_requests = []
       @previous_number_of_pull_requests = 0
+      @last_updated = nil
       @state_lock = Mutex.new
+      @last_updated_access_lock = Mutex.new
+      @force_update = true
+      @force_update_access_lock = Mutex.new
+    end
+
+    def last_updated
+      current_last_updated = nil
+      @last_updated_access_lock.synchronize {
+        current_last_updated = @last_updated.dup
+      }
+      current_last_updated
+    end
+
+    def force_refresh
+      do_force_update = false
+      @force_update_access_lock.synchronize {
+        do_force_update = @force_update
+      }
+      do_force_update
+    end
+
+    def force_refresh=(force_update)
+      @force_update_access_lock.synchronize {
+        @force_update = force_update
+      }
     end
 
     def setup
@@ -29,7 +56,7 @@ module Pra
     end
 
     def fetch_failed
-      output_string(4, 0, "Failed to fetch pull requests on 1 or more pull sources. Check #{Pra::Config.error_log_path} for details.")
+      output_string(4, 0, "Failed to fetch pull requests on 1 or more pull sources. Check #{Pra::Config.log_path} for details.")
     end
 
     def refresh_pull_requests(pull_requests)
@@ -37,6 +64,7 @@ module Pra
 
       @state_lock.synchronize {
         @current_pull_requests = pull_requests.dup
+        @last_updated = Time.now
       }
       draw_current_pull_requests
     end
@@ -51,10 +79,16 @@ module Pra
         when 'k', Curses::Key::UP
           move_selection_up
           draw_current_pull_requests
+        when 'r'
+          @force_update = true
         when 'o', ENTER_KEY
           @state_lock.synchronize {
-            Launchy.open(@current_pull_requests[@selected_pull_request_index].link)
+            Launchy.open(@current_pull_requests[selected_pull_request_loc].link)
           }
+        when 'n'
+          load_next_page
+        when 'p'
+          load_prev_page
         end
         c = Curses.getch()
       end
@@ -82,57 +116,125 @@ module Pra
     end
 
     def output_highlighted_string(row, col, str)
-      Curses.attron(Curses.color_pair(Curses::COLOR_CYAN)|Curses::A_NORMAL) {
+      Curses.attron(Curses::A_REVERSE) {
         output_string(row, col, str)
       }
     end
 
     def display_instructions
       output_string(0, 0, "Pra: Helping you own pull requests")
-      output_string(1, 0, "quit: q, up: k|#{"\u25B2".encode("UTF-8")}, down: j|#{"\u25BC".encode("UTF-8")}, open: o|#{"\u21A9".encode("UTF-8")}")
+      output_string(1, 0, "quit: q, up: k|#{"\u25B2".encode("UTF-8")}, down: j|#{"\u25BC".encode("UTF-8")}, open: o|#{"\u21A9".encode("UTF-8")}, refresh: r, next page: n, prev page: p")
     end
 
     def move_selection_up
       @state_lock.synchronize {
-        if @selected_pull_request_index > 0
-          @selected_pull_request_index -= 1
+        if @selected_pull_request_page_index > 0
+          @selected_pull_request_page_index -= 1
         end
       }
     end
 
     def move_selection_down
       @state_lock.synchronize {
-        if @selected_pull_request_index < @current_pull_requests.length-1
-          @selected_pull_request_index += 1
+        if selected_pull_request_loc < @current_pull_requests.length - 1 &&
+          (LIST_START_LINE + @selected_pull_request_page_index + 1) < Curses.lines
+          @selected_pull_request_page_index += 1
         end
       }
     end
 
+    def selected_pull_request_loc
+      (@current_page - 1) * pull_requests_per_page + @selected_pull_request_page_index
+    end
+
     HEADER_LINE = 6
     LIST_START_LINE = HEADER_LINE + 2
 
+    def columns
+      [
+        { name: :repository, size: 28, padding: 2 },
+        { name: :title, size: 45, padding: 2 },
+        { name: :author, size: 14, padding: 2 },
+        { name: :assignee, size: 14, padding: 2 },
+        { name: :labels, size: 12, padding: 2 },
+        { name: :updated_at, size: 16, padding: 2 }
+      ]
+    end
+
+    def header_width
+      @header_width ||= columns.reduce(0) do |t,c|
+        c[:size] + c[:padding] + t
+      end
+    end
+
+    def headers
+      header = ""
+      columns.each do |column|
+        header << "#{column[:name]}"
+        header << (" " * (column[:size] - column[:name].length))
+        header << " " * column[:padding]
+      end
+
+      header
+    end
+
+    def load_next_page
+      if @current_page + 1 <= pull_request_pages
+        @current_page += 1
+        clear_pull_requests
+        @selected_pull_request_page_index = 0
+        draw_current_pull_requests
+      end
+    end
+
+    def load_prev_page
+      if @current_page - 1 > 0
+        @current_page -= 1
+        clear_pull_requests
+        @selected_pull_request_page_index = 0
+        draw_current_pull_requests
+      end
+    end
+
+    def pull_requests_per_page
+      Curses.lines - LIST_START_LINE
+    end
+
+    def pull_request_pages
+      (@current_pull_requests.length.to_f/pull_requests_per_page).ceil
+    end
+
+    def clear_pull_requests
+      (LIST_START_LINE..Curses.lines).each do |i|
+        Curses.setpos(i, 0)
+        Curses.clrtoeol
+      end
+      Curses.refresh
+    end
+
     def draw_current_pull_requests
       @state_lock.synchronize {
-        output_string(3, 0, "#{@current_pull_requests.length} Pull Requests")
-        output_string(HEADER_LINE, 0, "repository      title                   from_reference          to_reference            author                  assignee                service")
-        output_string(HEADER_LINE + 1, 0, "-----------------------------------------------------------------------------------------------------------------------------------------------")
+        output_string(3, 0, "#{@current_pull_requests.length} Pull Requests @ #{@last_updated} : Page #{@current_page} of #{pull_request_pages}")
+        output_string(HEADER_LINE, 0, headers)
+        output_string(HEADER_LINE + 1, 0, "-" * header_width)
 
+        # clear lines that should no longer exist
         if @previous_number_of_pull_requests > @current_pull_requests.length
           start_line_of_left_overs = LIST_START_LINE+@current_pull_requests.length
-          last_line_of_left_overs = LIST_START_LINE+@previous_number_of_pull_requests - 1
-          (start_line_of_left_overs..last_line_of_left_overs).each do |i|
+          (start_line_of_left_overs..Curses.lines).each do |i|
             Curses.setpos(i, 0)
             Curses.clrtoeol
           end
           Curses.refresh
         end
 
-        @current_pull_requests.each_with_index do |pull_request, index|
+        # go through and redraw all the pull requests
+        @current_pull_requests[(@current_page-1)*pull_requests_per_page..@current_page*pull_requests_per_page-1].each_with_index do |pull_request, index|
           pull_request_presenter = Pra::CursesPullRequestPresenter.new(pull_request)
-          if index == @selected_pull_request_index
-            output_highlighted_string(LIST_START_LINE + index, 0, "#{pull_request_presenter.repository}\t#{pull_request_presenter.title}\t#{pull_request_presenter.from_reference}\t#{pull_request_presenter.to_reference}\t#{pull_request_presenter.author}\t#{pull_request_presenter.assignee}\t#{pull_request_presenter.service_id}")
+          if index == @selected_pull_request_page_index
+            output_highlighted_string(LIST_START_LINE + index, 0, pull_request_presenter.present(columns))
           else
-            output_string(LIST_START_LINE + index, 0, "#{pull_request_presenter.repository}\t#{pull_request_presenter.title}\t#{pull_request_presenter.from_reference}\t#{pull_request_presenter.to_reference}\t#{pull_request_presenter.author}\t#{pull_request_presenter.assignee}\t#{pull_request_presenter.service_id}")
+            output_string(LIST_START_LINE + index, 0, pull_request_presenter.present(columns))
           end
         end
       }

data/lib/pra/github_pull_source.rb

@@ -1,43 +1,119 @@
 require 'pra/pull_source'
 require 'pra/pull_request'
+require 'pra/log'
 require 'json'
 require 'faraday'
 
 module Pra
   class GithubPullSource < Pra::PullSource
+    def initialize(config = {})
+      @ratelimit_remaining = 5000
+      @ratelimit_limit = 5000
+      @ratelimit_reset = nil
+      super(config)
+    end
+
     def pull_requests
-      requests = []
-      repositories.each do |repo_config|
-        requests.concat(get_repo_pull_requests(repo_config))
+      return get_all_pull_requests
+    end
+
+    def fetch_pull_requests
+      pull_requests_json = "[]"
+      conn = Faraday.new
+      conn.basic_auth(@config['username'], @config['password'])
+      resp = conn.get do |req|
+        req.url rest_api_search_issues_url
+        req.params['q'] = "is:pr is:open sort:updated-desc #{repos_for_query}"
+        req.params['per_page'] = '300'
+        req.headers['Content-Type'] = 'application/json'
+        req.headers['Accept'] = 'application/json'
       end
-      return requests
+
+      @ratelimit_reset = Time.at(resp.headers['x-ratelimit-reset'].to_i)
+      @ratelimit_limit = resp.headers['x-ratelimit-limit'].to_i
+      @ratelimit_remaining = resp.headers['x-ratelimit-remaining'].to_i
+      Pra::Log.debug("Fetched pull requests and updated ratelimit tracking")
+      Pra::Log.debug("Ratelimit Reset: #{@ratelimit_reset}")
+      Pra::Log.debug("Ratelimit Limit: #{@ratelimit_limit}")
+      Pra::Log.debug("Ratelimit Remaining: #{@ratelimit_remaining}")
+      pull_requests_json = resp.body
+      Pra::Log.debug(pull_requests_json)
+      JSON.parse(pull_requests_json)
     end
 
-    def repositories
-      @config["repositories"]
+    def get_all_pull_requests
+      pull_requests = []
+
+      pull_requests_hash = fetch_pull_requests
+      pull_requests_hash['items'].each do |request|
+        begin
+          org, repository = extract_repository_from_html_url(request['html_url'])
+          unless excluded?(org, repository)
+            pull_requests << Pra::PullRequest.new(title: request["title"],
+                                                  from_reference: "",
+                                                  to_reference: "",
+                                                  author: request["user"]["login"],
+                                                  assignee: request["assignee"] ? request["assignee"]["login"] : nil,
+                                                  link: request['html_url'],
+                                                  service_id: 'github',
+                                                  repository: repository,
+                                                  updated_at: request["updated_at"],
+                                                  labels: request["labels"].collect{|l| l["name"]}.join(","))
+          end
+        rescue StandardError => e
+          Pra::Log.error("Error: #{e.to_s}")
+          Pra::Log.error("Request: #{request.inspect}")
+        end
+      end
+      pull_requests
     end
 
-    def get_repo_pull_requests(repository_config)
-      requests = []
-      JSON.parse(rest_api_pull_request_resource(repository_config)).each do |request|
-        requests << Pra::PullRequest.new(title: request["title"], from_reference: request["head"]["label"], to_reference: request["base"]["label"], author: request["user"]["login"], assignee: request["assignee"] ? request["assignee"]["login"] : nil, link: request['html_url'], service_id: 'github', repository: repository_config["repository"])
+    def repos_for_query
+      query_params = []
+      repositories.each do |repo|
+        query_params << "repo:#{repo['owner']}/#{repo['repository']}"
+      end
+
+      @excluded_repos = {}
+      organizations.each do |org|
+        query_params << "org:#{org['name']}"
+        @excluded_repos[org['name'].downcase] = org['exclude']
       end
-      return requests
+
+      return query_params.join(" ")
     end
 
-    def rest_api_pull_request_url(repository_config)
-      "#{@config['protocol']}://#{@config['host']}/repos/#{repository_config["owner"]}/#{repository_config["repository"]}/pulls"
+    def excluded_repos
+      @excluded_repos || collect_exclusions
     end
 
-    def rest_api_pull_request_resource(repository_config)
-      conn = Faraday.new
-      conn.basic_auth(@config['username'], @config['password'])
-      resp = conn.get do |req|
-        req.url rest_api_pull_request_url(repository_config)
-        req.headers['Content-Type'] = 'application/json'
-        req.headers['Accept'] = 'application/json'
+    def collect_exclusions
+      @exclusions = {}
+      organizations.each do |org|
+        @exclusions[org['name'].downcase] = org['exclude']
       end
-      resp.body
+      @exclusions
+    end
+
+    def excluded?(org, repository)
+      excluded_repos[org.downcase] && excluded_repos[org.downcase].include?(repository)
+    end
+
+    def extract_repository_from_html_url(html_url)
+      /https:\/\/github.com\/(\w+)\/([\w-]+)/.match(html_url)
+      return $1, $2
+    end
+
+    def rest_api_search_issues_url
+      "#{@config['protocol']}://#{@config['host']}/search/issues"
+    end
+
+    def repositories
+      @config["repositories"] || []
+    end
+
+    def organizations
+      @config["organizations"] || []
     end
   end
 end

data/lib/pra/log.rb

@@ -1,11 +1,36 @@
+require 'logger'
 require 'pra/config'
 require 'date'
 
 module Pra
   class Log
-    def self.log(message)
-      File.open(Pra::Config.log_path, 'a') do |f|
-        f.puts("#{DateTime.now.iso8601} - #{message}")
+    def self.logger
+      @logger ||= begin 
+        logger = Logger.new(Pra::Config.log_path, 10, 5000000)
+        logger.formatter = proc { |severity, datetime, progname, msg|
+          "#{datetime.iso8601} #{severity} - #{msg}\n"
+        }
+        logger.level = Logger::INFO
+        logger
+      end
+    end
+
+    def self.level(level)
+      logger.level = Logger.const_get level.upcase
+    end
+
+    def self.info(message)
+      logger.info(message)
+    end
+
+    def self.debug(message)
+      logger.debug(message)
+    end
+    
+    def self.error(message)
+      logger.error(message)
+      if message.respond_to?(:backtrace)
+        message.backtrace.each { |line| logger.error(line) }
       end
     end
   end

data/lib/pra/pull_request.rb

@@ -2,7 +2,8 @@ require 'pra/config'
 
 module Pra
   class PullRequest
-    attr_accessor :title, :from_reference, :to_reference, :author, :assignee, :link, :service_id, :repository
+    attr_accessor :title, :from_reference, :to_reference, :author, :assignee,
+      :link, :service_id, :repository, :labels, :updated_at
 
     def initialize(attributes={})
       @title = attributes[:title]
@@ -13,6 +14,8 @@ module Pra
       @link = attributes[:link]
       @service_id = attributes[:service_id]
       @repository = attributes[:repository]
+      @labels = attributes[:labels] || []
+      @updated_at = DateTime.parse(attributes[:updated_at]) unless attributes[:updated_at].nil?
     end
   end
 end

data/lib/pra/pull_request_service.rb

@@ -1,4 +1,4 @@
-require 'pra/config'
+require 'pra'
 require 'pra/pull_source_factory'
 require 'pra/pull_request_service/fetch_status'
 
@@ -18,13 +18,12 @@ module Pra
     end
 
     def self.pull_sources
-      config = Pra::Config.load_config
-      return map_config_to_pull_sources(config)
+      return map_config_to_pull_sources
     end
 
-    def self.map_config_to_pull_sources(config)
+    def self.map_config_to_pull_sources
       sources = []
-      config.pull_sources.each do |pull_source_config|
+      Pra.config.pull_sources.each do |pull_source_config|
         sources << Pra::PullSourceFactory.build_pull_source(pull_source_config)
       end
       return sources

data/lib/pra/version.rb

@@ -1,3 +1,3 @@
 module Pra
-  VERSION = "1.7.2"
+  VERSION = "2.0.0"
 end

data/lib/pra.rb

@@ -1,5 +1,7 @@
 require "pra/version"
 
 module Pra
-  # Your code goes here...
+  def self.config
+    @config ||= Pra::Config.load_config
+  end
 end

data/pra.gemspec

@@ -21,10 +21,11 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "bundler", "~> 1.10"
   spec.add_development_dependency "rake", "~> 10.1"
-  spec.add_development_dependency "rspec", "~> 2.14"
+  spec.add_development_dependency "rspec", "~> 3.5"
   spec.add_development_dependency "timecop", "~> 0.8"
 
   spec.add_dependency "faraday", "~> 0.9"
   spec.add_dependency "launchy", "~> 2.4"
   spec.add_dependency "curses", "~> 1.0"
+  spec.add_dependency "time-lord", "~> 1.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pra
 version: !ruby/object:Gem::Version
-  version: 1.7.2
+  version: 2.0.0
 platform: ruby
 authors:
 - Andrew De Ponte
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-12-04 00:00:00.000000000 Z
+date: 2016-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.14'
+        version: '3.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.14'
+        version: '3.5'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -108,6 +108,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: time-lord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 description: Command Line utility to make you aware of open pull-requests across systems
   at all times.
 email:
@@ -133,7 +147,6 @@ files:
 - lib/pra/config.rb
 - lib/pra/curses_pull_request_presenter.rb
 - lib/pra/curses_window_system.rb
-- lib/pra/error_log.rb
 - lib/pra/github_pull_source.rb
 - lib/pra/log.rb
 - lib/pra/pull_request.rb

data/lib/pra/error_log.rb

@@ -1,13 +0,0 @@
-require 'pra/config'
-require 'date'
-
-module Pra
-  class ErrorLog
-    def self.log(error)
-      File.open(Pra::Config.error_log_path, 'a') do |f|
-        f.puts("#{DateTime.now.iso8601} - #{error.message}")
-        error.backtrace.each { |line| f.puts(line) }
-      end
-    end
-  end
-end