smccarthy788 commented on PR #86: URL: https://github.com/apache/otava/pull/86#issuecomment-3201273482
> For the nested settings, I would say don't be afraid to just flatten or otherwise change the structure of the config file. Personally I quite like the nested structure of the config. IMO it's worth the custom parser. > I think there was something in the yaml configuration files - maybe it was the template system and how you configured each test... Where I felt that is probably best left as it is, "yaml only". I don't see it in this patch, so maybe that's because you likewise left those out of scope for this patch? I haven't touched how the `tests` are generated from the `templates` (I think this is what you're referring to). Still "yaml only", even more so now that I've removed the `expandvars` (no more arbitrary env vars). I have an itch to separate those from the config, but that would probably be best as a separate PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
