// Top matches · 7 engineers

Edaly Ahmed
@tomorro · Paris, France
81% MATCH
944 MERGED PRSDEVELOPER COMMUNICATION STYLE

WHY THEM  Writes about developer communication style in their own reviews.

“The `handleReviewClick` callback deps look correct. One thing — `router` and `transitionRouter` are both in the deps array which is fine, but it would be nice to double-check that `useTransitionRouter` from `next-view-transitions` is stable across renders (i.e. doesn't cause unnecessary re-creation of the callback). AFAIK it should be but worth confirming.”
Go deeper
73% MATCH
895 MERGED PRSCOMMUNICATION STYLE AND TONE

WHY THEM  Writes about communication style and tone in their own reviews.

“The goal here is to react to both `isPageFocused` and `hasNew` so making it an effect is the easiest way to do that. We might want to refactor polling more generally and move some of this state around but I'm hesitant to do this now.”
Go deeper
22% MATCH
36 MERGED PRSSIMON RABOURG COMMUNICATION STYLE

WHY THEM  Writes about simon rabourg communication style in their own reviews.

“The data that does change (legalAnalysisData, reviewData, citations, recommendations, summary) is not in the dependency array. So React sees identical deps, skips re-running the effect, and there's no duplicate execution.”
Go deeper
Load 1 more perspectives

People also asked