// Top matches · 6 engineers

100% MATCH
895 MERGED PRS

WHY THEM  Has reviewed this kind of change before.

“This is confusing. You need to walk through these stages: - Passing a ref to `<input>` - Creating your own component — it doesn't work anymore - Fixing it by spreading props Spreading props isn't a default thing people would reach for. The paragraph above doesn't match the code below because the code below introduces spreading.”
Go deeper
Edaly Ahmed
@tomorro · Paris, France
6% MATCH
944 MERGED PRSPROBLEM DESCRIPTION STYLE

WHY THEM  Writes about problem description style in their own reviews.

“This is the first cleanup step. A deeper refactor will come in a next PR. Context is big, and rewriting all identity usages in one go would make the PR massive it’s already quite heavy with just these minimal changes”
Go deeper
Load 3 more perspectives

People also asked