Skip to content

nmanoogian/review-tags

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

26 Commits
 
 
 
 

Repository files navigation

review-tags

This tool is built for teams who force-push changes to feature branches during PR review. It can help you keep track of code versions you've reviewed by creating tags whenever you've completed a review or re-review.

Installation

Put review-tags.sh in your path somewhere. Some prefer a short alias (e.g. rt) others prefer to hook it up as a custom git command (e.g. including it in your path as git-review allows you to invoke the script with git review).

Usage and Flow

When you start a review, you can checkout the branch manually or let the script do it for you with rt pull my-feature.

Note: The pull command will perform a git fetch.

You'll notice that the script creates a new tag named review/origin/my-feature/1, which marks the specific commit that you reviewed.

When the author force-pushes a change, you can run rt pull (no arguments implies current branch) to fetch the latest commit, update your local branch to this new commit, and create a new review tag (e.g. review/origin/my-feature/2).

You can view a plain diff of the change by doing rt diff 1 2 or simply rt diff which implies the last and second-to-last tags.

Often, a plain diff isn't valuable because the author might have rebased on the base branch. It's often more valuable to do rt range-diff to show a range-diff between the commits. This is effectively a "diff-diff" which shows what the author has changed in their own commits.

Lazygit Custom Command Example

This command pulls the locally checked out branch and shows the output in a CUI popup.

- key: "v"
  description: "rt pull"
  command: "rt p"
  context: "localBranches"
  showOutput: true

This command presents a menu to choose the rt diff command:

- key: "<c-v>"
  prompts:
    - type: 'menu'
      title: 'rt action'
      options:
        - name: 'range-diff'
          value: 'range-diff'
        - name: 'diff'
          value: 'diff'
  command: "rt {{index .PromptResponses 0}}"
  context: 'localBranches'
  subprocess: true

About

No description, website, or topics provided.

Resources

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published

Languages