Add a function to only freeze (if enabled) about immutableassign HOT 6 CLOSED

Feel free to use DeepFreeze instead...

from immutableassign.

I did something like this:

import iassign from 'immutable-assign'
import DeepFreeze from 'deep-freeze'

export const deepFreeze = obj => iassign.freeze ? DeepFreeze(obj) : obj

Would you be opposed to a PR with such a wrapper included in ImmutableAssign?

from immutableassign.

Why create dependency on DeepFreeze, when benefit (freezing is usefull for developer only, in production it is no-no) is so small ? Moreover you can add such trival wrapper to your project as single function without forcing all consumers of iassign to bloat their code with DeepFreeze?

from immutableassign.

Now I see that DeepFreeze is in current version required anyway...

from immutableassign.

Hi geirsagberg, I am not a fan of re-exporting of another module, which is antipattern. The better way is that you create your own module (called immutable-util for example), which export iassign() and deepFreeze(). And make your application codes depending on the new module, so you can swap out iassign() in the future if needed.

from immutableassign.

from immutableassign.