-
Notifications
You must be signed in to change notification settings - Fork 7
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
deepFreeze don't freeze non enumerable props #33
Comments
Freezing of non enumerable properties is available from v3.0.0 Major version has been published because as side effect option to deepFreeze prototype chain has also been removed (braking change). |
|
I will review it one more time and and consider bringing it back if I find some nice way of avoiding side effects (freezing built in JavaScript objects and their properties). |
As I see these side effects can be avoided by stopping prototype chain on one having This way if you freeze an array look my get-props for example. |
@astur Thanks for information i will investigate it. I'm also open to contributions if you want to contribute to it? |
Looks safe enough. Good. |
If object contains non enumerable prop with object value, it will not be frozen by deepFreeze.
The text was updated successfully, but these errors were encountered: