search

lundybernard / batconf

Application configuration tool from the BAT project
MIT License
2 stars 0 forks source link

Investigate DataclassConfig.get return values #27

Open lundybernard opened 3 months ago

lundybernard commented 3 months ago

I have some concerns about this return value.

https://github.com/lundybernard/batconf/blob/ffcf2fe4e9bdbd83c95853fd8a065bc815e75ecb/batconf/sources/dataclass.py#L27-L42

It has the potential to return a Node (DataclassConfig) instead of a Leaf (str), which could cause unexpected behavior.

This needs further investigation.