search

mantil-io / mantil

Build your AWS Lambda-based Go backends quicker than ever
https://www.mantil.com
MIT License
109 stars 3 forks source link

CLI commands outputs #82

Closed ksokolic closed 2 years ago

ksokolic commented 2 years ago

Adjust CLI outputs as defined in GoogleDocs.

ksokolic commented 2 years ago

After reviewing the first implemented version here are a few comments:

  1. We are removing *comments from the beginning, those that we are written in mantil aws install/uninstall and mantil stage new. These commands are often followed by prompt and I must agree with @IvanVlasic that reading a comment between operational information requires extra mental effort.
  2. We are keeping comments at the end of the command output as those are logically separated. However, when a comment is presented one extra line (prored) should be added at the end of the command output.
  3. Resources destroyed on was uninstall should be presented with - (minus). With aws uninstall we are destroying resources and with aws install we are creating those (and that's why we are using +). aws_uninstall
  4. Stage destroy is missing confirmation text and one extra line between prompt and Destroying... stage_destroy
  5. Stage new is not according to proposal in GoogleDoc + we are renaming Public API to Endpoint mantil_stage_new