joe-at-startupmedia / pmon3

Golang Production Process Manager
10 stars 0 forks source link

refactoring AppsConfig and ExecFlags #42

Closed joe-at-startupmedia closed 1 month ago

joe-at-startupmedia commented 1 month ago

Currently some previous architectural decisions have resulted in convolution that has spread from the system to configuration declarations and documentation. It was mainly caused by the exec command taking the process file as the first argument followed by flag specification. This has resulted in the process file being passed around separately from the exec flags throughout the entire codebase and resulted in an unnecessary AppsConfigApp struct to deal with configurations. All of this can be cleaned up by implementing the following rules: 1) Add File to the ExecFlags struct and refactor all calls which require both to only accept the ExecFlags struct. 2) Remove AppsConfigApp and replace with ExecFlags 3) Rename ever single instance of app/application to process

joe-at-startupmedia commented 1 month ago

released in v1.17.0