Skip to content
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

Use sindresorhus/node-defaults instead of deep-merge #19

Closed
benasher44 opened this issue Jul 31, 2024 · 1 comment
Closed

Use sindresorhus/node-defaults instead of deep-merge #19

benasher44 opened this issue Jul 31, 2024 · 1 comment

Comments

@benasher44
Copy link

sindresorhus/node-defaults already has checks to prevent prototype pollution. Switching to that may be easier than patching deep-merge

@75lb 75lb closed this as completed in ee28d6d Jul 31, 2024
@75lb
Copy link
Owner

75lb commented Jul 31, 2024

I factored deep-merge out of the project, replacing it with hand-rolled code. Released in table-layout v4.1.1. Let me know if there's anything else, thanks.

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

No branches or pull requests

2 participants