Los tips de code review de PostHog: 30 líneas o menos
PostHog publicó una lista práctica de tips de code review — las que de verdad usás en vez de las que fingís que leés. La idea: mantener los PRs bajo 300 líneas, escribir el commit message antes del código, y destacar los cambios riesgosos de forma explícita. Es una hoja de referencia, no un manifiesto.
Las que valen la pena son las aburridas. Un título y descripción claros ahorran más tiempo que cualquier linter. Los comentarios del reviewer deben ser accionables — no "esto se ve raro" sino "esta función toca la DB en un loop". Y si sos el que escribe el PR, resistí la tentación de defender el código en el thread; simplemente arreglá lo que pide el reviewer y subí otra vez.
Por qué nos importa: es el tipo de disciplina que mantiene a los equipos chicos sin ahogarse en bloat — y la que podés robarle a cualquiera, sin necesidad de ese brillo de Silicon Valley.
“Escribí el commit message antes del código.”