-
Notifications
You must be signed in to change notification settings - Fork 948
Tree-Shake Path #4477
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
Tree-Shake Path #4477
Conversation
|
Binary Size ReportAffected SDKs
Test Logs
|
@@ -215,11 +223,11 @@ export const validateFirebaseMergePaths = function ( | |||
// Check that update keys are not descendants of each other. | |||
// We rely on the property that sorting guarantees that ancestors come | |||
// right before descendants. | |||
mergePaths.sort(Path.comparePaths); | |||
mergePaths.sort((l, r) => pathCompare(l, r)); |
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.
does pathCompare
not work by itself?
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.
It should, but it might be a bit of an anti-pattern, as it might introduce some future breakages if the callsite ends up passing arguments that pathCompare
does not support (e.g. a "position" argument that is passed pathCompare
, which might expect an optional string description instead).
I did update this location though since it matches the original implementation.
Size Analysis ReportAffected Products |
No description provided.