fortify / fcli

fcli is a command-line utility for interacting with various Fortify products
https://fortify.github.io/fcli/
Other
31 stars 17 forks source link

Reconsider `fcli state` structure #357

Closed rsenden closed 1 year ago

rsenden commented 1 year ago

It seems a bit overkill to have a top-level fcli state module with only a single variable sub-entity. Variable-related functionality would also fit perfectly well in the util module.

So, to keep the fcli command tree more concise, proposal is to move the state commands as follows:

The only potential drawback is that if we ever want to have additional state-related commands (although I can't think of any right now), these would likely also need to go into the util module, or we might need to revert to having a dedicated state module.

Ideas/suggestions/...?

psmf22 commented 1 year ago

agree with moving it to util, a separate module for 2 commands seems overblown

rsenden commented 1 year ago

Others also seem to agree that moving to util would be better; shouldn't be too much effort so will implement this.