Move new app initialization out of main#178
Conversation
|
@tomoyamachi can you please have a look? |
|
@lior-orca Apologize for the delayed response. Line 28 in f39e932 How do you think about it? |
|
@tomoyamachi I think it will be over-engineering since any new command-line option will need to be added to in both places (the new struct and in the cli.Context definition) I tried to have something similar to trivy Let me know what you think |
|
@tomoyamachi see my above replay |
|
@tomoyamachi one more try. |
|
@lior-orca Apologize for the delayed response. I merged it. |
|
@tomoyamachi thanks! |
In order to use 'Dockle' programmatically, the app initialization logic cannot be placed in the main package.
The commit simply transitions the relevant logic out of the main package.