Skip to content

stevegrunwell/code-review-for-me-and-you

Repository files navigation

Code Review: For Me & You

On the surface, the idea of code review is a no-brainer: why wouldn't we want a second set of eyes on our code, especially before deploying to production?

As we peel back the layers, however, we find that the topic of code review is much more nuanced. How detailed should the review be? Who is qualified to perform the review (hint: it's not just senior developers)? Can we afford to take another developer away from their project to review this one? What steps can we take to ensure reviews are constructive, rather than demoralizing?

Attendees will gain deeper insight into some of the arguments for and against systemic, peer code review, as well as pick up some useful tools to make code review a natural part of their teams' workflow.

View slides

Presentation History

Resources

Below are links to some of the tools referenced throughout the talk.

Linting & Static Code Analysis Tools

  • PHP_CodeSniffer — Extremely popular linter (and fixer) for PHP
  • PHP Coding Standards Fixer — Alternative to PHP_CodeSniffer from the Symfony ecosystem
  • PHPStan - A very popular—and dare I say the best?—static code analysis tool for PHP
  • Psalm - Another popular PHP static code analysis tool (from Vimeo)
  • ESLint - Prominent linter for the JavaScript ecosystem
  • Shellcheck - Shell script analysis tool
  • RuboCop - Linter/formatter for Ruby
  • PyLint - Linter for Python code

Git Hooks

  • Husky - Tool that can automatically install Git hooks in your local repo
  • Lefthook - Alternative to Husky, written in Go

About

Slides for "Code Review: For Me & You"

Resources

Stars

Watchers

Forks

Packages

No packages published