nbdev icon indicating copy to clipboard operation
nbdev copied to clipboard

properly escape json and yaml on `nbdev_new`

Open ozpau opened this issue 9 months ago • 2 comments

Fixes #1493

ozpau avatar Feb 06 '25 17:02 ozpau

@jph00 I am not sure about this one, do we want to sanitize double quotes etc?

hamelsmu avatar Feb 27 '25 06:02 hamelsmu

@jph00 I am not sure about this one, do we want to sanitize double quotes etc?

tbh I never use that CLI command, and I didn't write it, so I don't know! Sounds like maybe it is required for it to work correctly? Maybe you could give this PR a quick test to confirm it works OK for you?

jph00 avatar Feb 28 '25 06:02 jph00