-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
DataType's mutable field removal: put in NEWS.md or unintentional? #41049
Comments
Changes to internals are typically not given a NEWS entry. People that use them are sort of expected to be able to look up what happened to it and how to fix their code. |
I guess "people" in this case is Jeff 😂 JuliaIO/BSON.jl#63 |
In 1.5+ there is a function |
|
Thanks for the info! |
Up to the current 1.6.1 version,
DataType
s have a mutable fieldwhich seems to have gone in the 1.7 master branch:
This has caused, e.g. BSON.jl to fail due to https://github.com/JuliaIO/BSON.jl/blob/ecae1169be8fb20a88ed2d23417e8585e4a2a7fe/src/write.jl#L46
Is this change intentional (in which case this should probably put into NEWS.md/provide a current way to check if certain type is mutable via
ismutabletype
)?julia
versioninfo()
:The text was updated successfully, but these errors were encountered: