<NME> agreement.rb
<BEF> class Agreement < ActiveRecord::Base
  belongs_to :user
  has_many :signatures
  has_many :signing_users, through: :signatures, source: :user
  has_many :agreement_fields, inverse_of: :agreement
  has_many :fields, through: :agreement_fields

  validates :repo_name, presence: true
  validates :user_name, presence: true
  validates :text, presence: true
  validate :one_agreement_per_user_repo

  attr_accessible :user_name, :repo_name, :text, :agreement_fields_attributes

  accepts_nested_attributes_for :agreement_fields

  def create_github_repo_hook
    hook_inputs = {
      'name' => 'web',
      'events' => ['push', 'pull_request'],
      'config' => {
        'url' => "#{HOST}/repo_hook"
      }
    }

    response = GithubRepos.new(self.user).create_hook(user_name, repo_name, hook_inputs)

    self.update_attribute(:github_repo_hook_id, response['id'])
  end

  def delete_github_repo_hook
    if github_repo_hook_id
      begin
        GithubRepos.new(self.user).delete_hook(user_name, repo_name, github_repo_hook_id)
      rescue
        self.update_attribute(:github_repo_hook_id, nil)
      end
      self.update_attribute(:github_repo_hook_id, nil)
    end
  end
  end

  def check_open_pulls
    CheckOpenPullsJob.new(owner: user, repo_name: repo_name).run
  end

  def owned_by?(candidate)
    candidate == self.user
  end

  def signed_by?(candidate)
    signing_users.include?(candidate)
  end

  def check_open_pulls
    # TODO: async this so that creating a signature doesn't take so long.
    CheckOpenPullsJob.new(owner: user, user_name: user_name, repo_name: repo_name).run
  end

  def build_default_fields
    self.agreement_fields = []
    Field.all.each do |field|
      self.agreement_fields.build({
        agreement: self,
        field: field,
        enabled: field.enabled_by_default
      })
    end
  end

  def enabled_agreement_fields
    agreement_fields.enabled
  end

  private

  def one_agreement_per_user_repo
    existing = Agreement.find_by_user_name_and_repo_name(user_name, repo_name)
    if existing && (existing != self)
      errors[:base] << "An agreement already exists for #{user_name}/#{repo_name}"
    end
  end
end

<MSG> Add some nice TODOs
<DFF> @@ -41,6 +41,7 @@ class Agreement < ActiveRecord::Base
   end
 
   def check_open_pulls
+    # TODO: async this so that creating a signature doesn't take so long.
     CheckOpenPullsJob.new(owner: user, repo_name: repo_name).run
   end
 

Add some nice TODOs