Skip to content

Instantly share code, notes, and snippets.

@Carmer
Created May 31, 2016 03:28
Show Gist options
  • Select an option

  • Save Carmer/5102523a97e5bb177fa6b2fcd17af2ae to your computer and use it in GitHub Desktop.

Select an option

Save Carmer/5102523a97e5bb177fa6b2fcd17af2ae to your computer and use it in GitHub Desktop.

Mix Master Peer Review Session

This peer review session is aimed at practicing your ability to read code, communicate about your own and other peoples code, and give some practice in helping others troubleshoot their apps.

Things to cover:

  • First start with any issues or confusion the first partner may have had getting things to work. Try to troubleshoot things together if there is still a bug in your app. Here are some things you may want to consider:

    • The request response cycle
    • RSpec/testing
    • Capybara/feature testing
    • User stories
    • Conditionals in views
    • Strong params
    • current_user in the application controller.
    • Controllers
    • Models
    • Sessions
    • Git
    • Partials
    • Factory Girl
    • Oauth / Spotify
  • Take a look at the other persons code. Navigate your way through the code asking questions or for clarification when something is unclear. Are there any critiques you have? Don't be shy, this is a time to discuss and review code. Some things to look at are:

    • Ruby/Rails Style
    • Syntax
    • Variable or method naming
    • Following MVC principals like pushing logic down the stack.
    • Robust testing

Make sure you and your partner write down any questions you may have and bring them to the Mix Master Review session right after this peer review session.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment