• Google MT&E
Engineering Resources
  • Best practices
    The ideal place to start for answering the question of "How should I..."?
  • Frameworks
    The frameworks of choice for building, launching, and automating websites and applications
  • Infrastructure
    Reference material on configurations and pushes
  • Languages
    Information about Google Marketing's development languages of choice
  • Libraries
    Approved development libraries to expedite the build process
  • Platforms
    Foundations for applications, processes, and technologies to be devleoped
  • Tools
    Reference material on internal tooling
  • Administrative
    Internal-only reference documentation
Agency Guide
  • English
  • Bahasa Indonesia
  • Deutsch
  • Español
  • Français
  • Português – Brasil
  • Русский
  • 中文 – 简体
  • 日本語
  • 한국어
Sign in
  • Best Practices
  • Google MT&E
  • Engineering Resources
    • More
  • Agency Guide
  • Overview
  • Accessibility/A11y
    • Overview
    • Googlers
      • Overview
      • Teleport
  • APAC-specific information
    • Overview
    • Googlers
  • Code review guidelines
  • Gerrit workflow
  • Git workflow
  • Style Guides
    • HTML & CSS Style Guides
    • TS/JS Style Guide
    • Sass Style Guide
    • Public Style Guides
  • Images
    • Overview
    • Googlers
  • Localization/L10n
    • Overview
    • Googlers
      • Overview
      • Terminology
      • Per-Locale fixes
      • Placeholder text
      • Translation Request
      • SCS folder structure and fallbacks
      • Country top level domains
  • Linting/Formatting
    • Overview
    • Googlers
  • Marketing Web Standards
  • Performance
  • Practices
  • Privacy
  • QA
    • Overview
    • Googlers
      • Overview
      • Running a local SFFE
      • Resources
        • Overview
        • Quality Assurance
        • Testing strategies and testing planning
        • Guidelines for Updating Buganizer with QA Feedback
        • Cybage L10N QA Scope of testing
      • Testing Methods
        • Overview
        • Proxy testing: cURL
        • Testing local SFFE with FoxyProxy
        • Visual Regression Tests
          • Overview
          • Storing secret keys in keystore
      • VMWare Virtual Machines
      • WebPageTest
  • Readability
  • Security
    • Overview
    • Best Practices: Security
      • Overview
      • Cookie Policy
      • Review process
      • Testing
        • Overview
        • Running Inquisition
        • Inquisition Mass scanning (WMX)
  • SEO
  • Webmaster SLA
    • Overview
    • Googlers
  • YouTube Marketing Web Standards
  • Brand Studio UX Engineering
  • Best practices
  • Frameworks
  • Infrastructure
  • Languages
  • Libraries
  • Platforms
  • Tools
  • Administrative
  • Home
  • Google MT&E
  • Engineering Resources
  • Best Practices

Code review guidelines

Stay organized with collections Save and categorize content based on your preferences.

These guidelines help us as we grow by ensuring we don't encounter productivity obstacles that could've been avoided.

Learn more (Googlers) Learn more

All rights reserved. Java is a registered trademark of Oracle and/or its affiliates.

Last updated 2023-01-06 UTC.

  • Connect

    • Blog
    • Facebook
    • Medium
    • Twitter
    • YouTube
  • Programs

    • Women Techmakers
    • Google Developer Groups
    • Google Developers Experts
    • Accelerators
    • Google Developer Student Clubs
  • Developer consoles

    • Google API Console
    • Google Cloud Platform Console
    • Google Play Console
    • Firebase Console
    • Actions on Google Console
    • Cast SDK Developer Console
    • Chrome Web Store Dashboard
Google Developers
  • Android
  • Chrome
  • Firebase
  • Google Cloud Platform
  • All products
  • Terms
  • Privacy
  • Sign up for the Google Developers newsletter Subscribe
  • English
  • Bahasa Indonesia
  • Deutsch
  • Español
  • Français
  • Português – Brasil
  • Русский
  • 中文 – 简体
  • 日本語
  • 한국어