-
Notifications
You must be signed in to change notification settings - Fork 47k
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
[cleanup] remove flag enableAddPropertiesFastPath #31062
Conversation
The experiment was tested internally and rolled out, replacing the flag with `true`.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Comparing: f9ebd85...403bab0 Critical size changesIncludes critical production bundles, as well as any change greater than 2%:
Significant size changesIncludes any change greater than 0.2%: Expand to show
|
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.
Hmm... @dmytrorykun, can we also change addNestedProperty
to use fastAddProperties
(and then delete addProperties
and rename fastAddProperties
to addProperties
)?
Landing as we should run a different experiment if we also change the update path. |
The experiment was tested internally and rolled out, replacing the flag with
true
.