chore(SidebarLink): update effect deps
Change for:
- ref in deps array doesn't have any effect.
-
Boolean(ref)is always true.
Some other question:
As doc says, u should not suppress deps linter. In user code, it's easy to change code to fit it. However, checking the reference stability of a library hook's return value, especially when it's a function, can be challenging as it requires inspecting the source code. In some case, I have to suppress deps linter because libray code return a function that is not wrapped by useCallback and changes its reference on every render. I also want to know, even if the return value is stable like ref, should I still add it to deps list? Will it affect the future Forget compiler work? In usePendingRoutes, it seems omit events return by useRouter.
Size Changes
📦 Next.js Bundle Analysis
This analysis was generated by the next.js bundle analysis action 🤖
Three Pages Changed Size
The following pages changed size from the code in this PR compared to its base branch:
| Page | Size (compressed) | First Load |
|---|---|---|
/404 |
75.6 KB (-4 B) |
168.53 KB |
/500 |
75.59 KB (-4 B) |
168.53 KB |
/[[...markdownPath]] |
77.01 KB (-4 B) |
169.95 KB |
Details
Only the gzipped size is provided here based on an expert tip.
First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If next/link is used, subsequent page loads would only need to download that page's bundle (the number in the "Size" column), since the global bundle has already been downloaded.
Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis
Next to the size is how much the size has increased or decreased compared with the base branch of this PR. If this percentage has increased by 10% or more, there will be a red status indicator applied, indicating that special attention should be given to this.