-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
0.10.0 (2) - Warn if makeClient returns the same client for multiple requests. #205
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume that you haven't written tests because this is an [idea] right?
In general I like this idea a lot since it at least gives some indication of a potential footgun for those that are possibly unaware of the impact of returning the same client instance. Nice work!
Yup, I wanted to make sure we actually want to do this before I get into the nitty-gritty of figuring out how to write a test against a RSC environment :) |
4c1330e
to
407204a
Compare
491348d
to
4b786c2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had some small suggestions on wording, but this is great!
Co-authored-by: Jerel Miller <[email protected]>
0abc532
to
d06e3c2
Compare
8e0bb7c
to
d316a92
Compare
3ea0f36
to
ac208d7
Compare
Job #106: Bundle Size — 1.01MiB (0%).Warning Bundle contains 1 duplicate package – View duplicate packages Bundle metrics
|
Current Job #106 |
Baseline Job #104 |
|
---|---|---|
Initial JS | 890.59KiB |
890.59KiB |
Initial CSS | 0B |
0B |
Cache Invalidation | 0.04% |
0.04% |
Chunks | 24 |
24 |
Assets | 45 |
45 |
Modules | 512 |
512 |
Duplicate Modules | 30 |
30 |
Duplicate Code | 1.29% |
1.29% |
Packages | 29 |
29 |
Duplicate Packages | 1 |
1 |
Bundle size by type no changes
Current Job #106 |
Baseline Job #104 |
|
---|---|---|
JS | 1023.85KiB |
1023.85KiB |
Other | 5.99KiB |
5.99KiB |
View job #106 report View pr/getClient-makeUniqueClients branch activity View project dashboard
size-limit report 📦
|
No description provided.