Immutable classes

[getzze]

closes #35

REBASED TO #49

Uses the attrs package to create frozen classes.
The new instances are picklable in an incompatible way (unpickling old classes would not work).

As a bonus, the instances can be unstructured using cattrs.

This is a test PR, maybe it makes more sense to keep the old code and just forbid setting the attributes using properties.

[Diaoul]

Thanks for the typing and work there! Should we stick to the stdlib and use dataclasses with frozen=True? My main concern is introducing a dependency on another package

[getzze]

Hi! There is no converter and validator in `dataclasses` that is why I did it with `attrs`. But I realized there is a way to do it anyway, so I will move it to `dataclasses` to avoid a dependency.

…

On 26 September 2024 08:09:20 GMT+01:00, Antoine Bertin ***@***.***> wrote: Thanks for the typing and work there! Should we stick to the stdlib and use dataclasses with frozen=True? My main concern is introducing a dependency on another package -- Reply to this email directly or view it on GitHub: #50 (comment) You are receiving this because you authored the thread. Message ID: ***@***.***>

[getzze]

I struggled a bit with using __getattr__ and pickle without defining __setstate__, but I solved it finally.

It uses only the dataclasses stdlib now. I left cattrs as a test dependency, but maybe it's not worth it either, let me know what you think.