Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Commits on Jun 15, 2010
  1. @woahdae

    Made review request emails contain commit message info

    woahdae authored
    This inserts the last commits subject into the email subject,
    and the body into the email body. This would help me to see
    what people are doing, connecting review emails with in-person
    discussions, etc
  2. @woahdae

    Fix "accept doesn't honor non-origin remotes"

    woahdae authored
    Accept should now honor non-origin remotes
Commits on May 12, 2010
Commits on Apr 29, 2010
  1. Make accepting changes rebase onto the current branch.

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  2. Accidentally broke request by not checking out the review branch

    authored
    In request it creates a branch for code review and adds a commit to
    it. By not checking it out after creating it the commit was added to
    whatever branch the review request was issued from. This also resulted
    in the wrong code being pushed for review
    
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  3. Send an email to the requester when the change is merged

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  4. Delete the remote review branch after it is merged

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  5. Reviewr pushes the changes to master after they are merged

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  6. Make accepting a review set merge in the different commits

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  7. Make accept check if the branch can be added cleanly to master

    authored
    Accept now will try to rebase the reviewed code onto the remote
    master. This will check for conflicts and ensure that the
    non-meta-data commits from the review branch can be cherry-picked onto
    the master branch
    
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  8. Add an accept command that only fetches the correct branch

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
  9. Extract a command superclass to prepare for adding more commands

    authored
    Signed-off-by: Ryan Burrows <rhburrows@gmail.com>
Commits on Apr 13, 2010
  1. Add error handling to the mailer

    authored
    Change the mailer so when the actual email fails (i.e. the username
    and password are wrong) an error message saying so is displayed to the
    user.
    
    Issue #3
  2. Change back to original branch after sending review request

    authored
    Change it from leaving the requester on the branch created for the
    request to changing back to the branch that the request was generated
    from. This should help avoid double review commits being accidentally
    in the history.
    
    From Issue #2
Commits on Apr 8, 2010
  1. Add support for using a different remote repository

    Ryan Burrows authored
  2. Polish up ui

    Ryan Burrows authored
  3. Fixing up gmail integration

    Ryan Burrows authored
  4. Finish up the request review command

    Ryan Burrows authored
  5. Add support for capturing user email and password.

    Ryan Burrows authored
    This information isn't used yet, just captured
Commits on Apr 6, 2010
Commits on Apr 3, 2010
Commits on Apr 1, 2010
  1. A little cleanup refactoring

    Ryan Burrows authored
Commits on Mar 31, 2010
  1. Reorganize code for future extension

    Ryan Burrows authored
  2. Add an empty help command for fall-through

    Ryan Burrows authored
  3. Change Reviewr.run to take an array instead of a string

    Ryan Burrows authored
    ARGV is the arguments to the command line in an array, not just a
    string so change the main entry point to match this
  4. Send review email

    Ryan Burrows authored
  5. push the branch to origin

    Ryan Burrows authored
  6. Add a metadata commit to the review branch

    Ryan Burrows authored
  7. Generate a branch for the code review request

    Ryan Burrows authored
  8. Set up cucumber and rspec

    Ryan Burrows authored
Something went wrong with that request. Please try again.