Force model to be object and always be immutable? about hyperapp HOT 13 CLOSED

Thanks for supporting Hyperapp, I like it too ;) but let's not derail this thread or any other to criticize choo. choo was a big source of inspiration for me.

With regards to the issue at hand, I have nothing against object.freeze, but I don't know if it is worth introducing in core.

from hyperapp.

There's a mixin solution for this: https://github.com/okwolf/hyperapp-freeze

from hyperapp.

@charlike yes @jbucaran is correct, hyperapp-freeze will get cleaned up after the hyperapp-logger refactor is complete.

from hyperapp.

@tunnckoCore What's the advantage?

from hyperapp.

what you think about the model to be object

choo does this IIRC. HyperApp is more flexible allowing the model to be a string, number, array or object.

from hyperapp.

choo does this IIRC

That's why I'm asking. I think it is immutable for reason. And that isn't only in choo, actually. "immutable" and "functional" are kinda trending these days.

I'm not so frontend, but I like that and I believe that's good and correct trend.

from hyperapp.

Can you freeze a number or an array or is it limited to objects?

What about simply not changing the model? The only reason you'd want to freeze the model is to avoid mutations, but you can also avoid this by not mutating the model yourself.

from hyperapp.

Can you freeze a number or an array or is it limited to objects?

No. In ES2015 it will return it, in below versions it will throw TypeError.

What about simply not changing the model?

For same reason that most devs use flow/typescript - guarantees, I kinda still don't like them, as I don't like the native assert and massive usage of it then "unassertifyfyfyfyfyf" (like Yosh and browserify & co devs does and promotes).

from hyperapp.

but let's not derail this thread or any other to criticize choo. choo was a big source of inspiration for HyperApp.

Yea, it is great in anyway, I just sharing my view points. :)

from hyperapp.

@tunnckoCore Feel like we can close this?

from hyperapp.

yea, it is clear :)

from hyperapp.

@okwolf that's cool idea, but any reasonable and logical reason to hardcode the deep-freeze package instead of including it as normally and expose cjs, es and umd bundles (with rollup for example, it's pretty easy)?

from hyperapp.

@charlike Yeah, I think he was in a hurry.

@okwolf This should be good to get you started.

• okwolf/hyperapp-logger#1

Same cosmetic changes apply to hyperapp-freeze, etc. 👋

from hyperapp.