// Top matches · 7 engineers

100% MATCH
895 MERGED PRSCODE PROTECTIVENESS AND OPENNESS TO REFACTORING

WHY THEM  Writes about code protectiveness and openness to refactoring in their own reviews.

“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
Sébastien Decrême
@tomorro · Paris
56% MATCH
943 MERGED PRSSMALL PRS, REAL DESCRIPTIONS, CURSOR ASSISTED SUMMARIES

WHY THEM  Writes about small prs, real descriptions, cursor assisted summaries in their own reviews.

“can you add it to the right issue / PR please, this way we will be able to track it and test it This part is critical, it implies a different scope, not only the platform squad and need some non regression tests Thanks 🙏”
Go deeper
Edaly Ahmed
@tomorro · Paris, France
10% MATCH
944 MERGED PRSCODE PROTECTIVENESS AND CONSISTENCY ENFORCEMENT

WHY THEM  Writes about code protectiveness and consistency enforcement in their own reviews.

“Having the arrange logic inside each describe (afterEach and beforeEach) gives clearer context for the assertions below. It keeps tests self-contained and easier to read without jumping around the file. If we hoist setup to the top, switching context becomes harder. Keeping it per describe feels more natural and scalable. Happy to align if you feel strongly though!”
Go deeper
Load 3 more perspectives

People also asked