Hi Ming, I updated based on most of your review except "I suggest use argparse.FileType for argument —manifest and --config, which will take care of
detecting file existence and permission etc."
The filetype is not only for validate the file path but also, read file content to _io.TextIOWrapper. Tried yaml.dump but not work. Since user could use config file only, we still need to validate and load the manifest file. We have no gain for such change. We may use it later in some pure arg app.
Hi Ming, I updated based on most of your review except "I suggest use argparse.FileType for argument —manifest and --config, which will take care of detecting file existence and permission etc."
The filetype is not only for validate the file path but also, read file content to _io.TextIOWrapper. Tried yaml.dump but not work. Since user could use config file only, we still need to validate and load the manifest file. We have no gain for such change. We may use it later in some pure arg app.