lesurp / OptionalStruct

Macro copying a struct with Option fields. Useful for config initialization
Apache License 2.0
36 stars 13 forks source link

Make can_convert public #27

Closed t-moe closed 2 months ago

t-moe commented 3 months ago

... so that you can make nested optional structs across different files.

(doing so leads to a compile error without this change)

lesurp commented 2 months ago

Hi, thank you very much for your contribution! I think it makes more sense however to move this method to the Applicable trait, as that makes it public but also allows using it in generic contexts (I don't know if that's a use-case, but I see no point in restricting it). You should already be able to pull the latest version :)