Skip to content

Conversation

@malua
Copy link

@malua malua commented Jun 15, 2025

Hey,
I would like to propose to replace lodash.isequal with fast-equals strictDeepEqual.
I was not entirely sure if deepEqual or strictDeepEqual would be the correct replacement.

I want to replace it because vite has issues with the import of lodash.isequal and the package is already deprecated.

@CLAassistant
Copy link

CLAassistant commented Jun 15, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@nachocodoner nachocodoner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think strictDeepEqual (strict equality (===) for primitive values, no type coercion) is the closest equivalent of Lodash's isEqual.

I accept this change, I will publish it soon. This will also make the runtime more performant in theory 🚀

image

Thanks for the contribution

@Grubba27 Grubba27 mentioned this pull request Oct 17, 2025
@Grubba27 Grubba27 changed the base branch from master to feature/4.0.1 October 17, 2025 14:31
@Grubba27 Grubba27 merged commit 7361bcc into meteor:feature/4.0.1 Oct 17, 2025
2 checks passed
@Grubba27
Copy link
Contributor

Published this under [email protected] please give a test and let me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants