diff --git a/REVIEW.md b/REVIEW.md index 312fcd20c5..3aaaae01c7 100644 --- a/REVIEW.md +++ b/REVIEW.md @@ -269,3 +269,15 @@ if (!document || duration <= 0) { return; } ``` + +## Static member functions use PascalCase + +Non-static member functions use camelCase (`startBatch`, `finalize`). Static member functions use PascalCase (`ShouldTrack`, `Parse`, `Create`), matching the convention for free functions. + +```cpp +// BAD - camelCase for static method: +[[nodiscard]] static bool shouldTrack(not_null item); + +// GOOD - PascalCase for static method: +[[nodiscard]] static bool ShouldTrack(not_null item); +```