Skip to content

Conversation

@mortenpi
Copy link
Member

If we happen to get nothings or other values that TOML.jl can not serialize in the the DataSet's .conf, the show() method will break printing in the REPL. So let's get rid of all problematic values. We don't care here that it is round-trippable -- this is just for a pretty show method.

In addition, there TOML.print's to_toml does not handle e.g. unsupported Dict keys. So we also just put a catchall try-catch around it and print something sensible, so that the users wouldn't be presented with a big show method stacktrace.

@mortenpi mortenpi requested a review from pfitzseb August 14, 2025 16:03
@mortenpi mortenpi merged commit 85c212d into master Aug 15, 2025
14 checks passed
@mortenpi mortenpi deleted the mp/filter-nothing branch August 15, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants