search

skybet / cali

Cali Automation Layout Initialiser
MIT License
32 stars 7 forks source link

Moved Command and Task out of cli.go #33

Closed edupo closed 6 years ago

edupo commented 6 years ago

The rebase of #29 was not clear to me so I just did the changes from master

wheresalice commented 6 years ago

LGTM and staticli still compiles with this change.

I'm new to Go, but if this was Ruby I'd be considering a lib directory and putting most of the code into there. Is that idiomatic Go? Or is putting all the go files in the root of the repo more sensible?

lucymhdavies commented 6 years ago

Just did some diffs, to check for typos etc., and it's exactly what it looks like: moving code from one big file into other smaller files. :)

lucli compiles too, so LGTM!