You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
YAML::Store dependency for pstore is required and fails on Ruby 4.0 #69
I'm not sure what is best: make yaml depend on pstore in the gemspec or simply make it a soft dependency. In either case, the LoadError shouldn't be silently supressed.
After thinking about it a little bit I also thought about extracting the store functionality would be the way to go.
After giving it more thought, I think it is fine to leave it as it is for the major part.
With Ruby 4 the warning disappears and an exception will be raised when the file is being loaded.
The only thing I'd change is to emit instructions when catching the LoadError but still let it fail with the exception.
Another option would be to include this into PStore instead of YAML. There, they can simply load Psych.
Activity
boof commentedon Jan 15, 2026
IMHO, this is a hard dependency and should be added to the gemspec. IDK the minimum version, though.
boof commentedon Jan 30, 2026
After thinking about it a little bit I also thought about extracting the store functionality would be the way to go.
After giving it more thought, I think it is fine to leave it as it is for the major part.
With Ruby 4 the warning disappears and an exception will be raised when the file is being loaded.
The only thing I'd change is to emit instructions when catching the LoadError but still let it fail with the exception.
Another option would be to include this into PStore instead of YAML. There, they can simply load Psych.