search

smilelikeshit / cli-start-stop-instance-aws-gcp-do

learning go to build CLI
0 stars 1 forks source link

you can use switch case and than you check `start` on lowercase and uppercase you can convert to lowercase or uppercase first #1

Closed Arif9878 closed 3 years ago

Arif9878 commented 3 years ago

https://github.com/smilelikeshit/cli-start-stop-instance-aws-gcp-do/blob/72e15e22f32975e32da457ef73555b9662d3363f/cmd/aws.go#L34

firmanJS commented 3 years ago

Yes I agree, it's better to convert to lowercase first or uppercase @smilelikeshit

smilelikeshit commented 3 years ago

https://github.com/smilelikeshit/cli-start-stop-instance-aws-gcp-do/pull/3

has alread fix by fritend @sandisunandar99 @firmanJS @Arif9878