chore: rewrite project structure#108
Conversation
hrodmn
left a comment
There was a problem hiding this comment.
@jjfrench thanks for taking this on! I have a few ideas for you to consider but don't have any very strong opinions here.
I think the most actionable one might be to consider breaking the big global Config class into smaller specific configs for various components. Right now we take the big Config and construct things like PgstacDbConfig directly from the Config properties. I think it would be cleaner and easier to maintain if we did not need to pass properties through the global Config to the lower level configs. That way if we change things or add a property we would only need to add it in one place rather than adding it to Config and adding it to XXXConfig.
We don't need to do it in this PR but it is probably time to break PgstacInfra into some smaller constructs! That thing is a beast.
Co-authored-by: Henry Rodman <henry.rodman@gmail.com>
hrodmn
left a comment
There was a problem hiding this comment.
Nice work! Thanks for working through the config details with me.
Description
Rewrites the project structure in python rather than TypeScript and Node - aims to not change any of the infrastructure
(would close open dependabot security issues for javascript)