Document weaknesses of config parser in python

mar-file-system / mad

MarFS Automatic Deployment

Other

0 stars 0 forks source link

Document weaknesses of config parser in python #10

Closed shanegoff closed 3 years ago

shanegoff commented 3 years ago

~~Only string values can be used for conversion to XML. If you set a value that looks like a number to an object of type int the xml process gets real sad~~
There can be no empty values in the config file. I don't see any reason for that to be problematic but issues could rise if we don't stick to that limitation. None type values result in the element being skipped entirely so you will be missing an element.

shanegoff commented 3 years ago

None types will now cause a sys exit of the parser any value assigned an int or float will be converted to str type before going to xml

shanegoff commented 3 years ago

Comments can be saved, but do not currently produce a readable file, which seems like a step back. Comments are not currently retained in the data bindings, and modification of the config file will be done in a different way.