Re: Traffic Ops code re-structure

2018-11-14 Thread Dan Kirkwood
I've always advocated keeping the main package lean from the start. +1 on making this change. -dan On Wed, Nov 14, 2018 at 10:06 AM Robert Butts wrote: > Well, when I originally wrote `traffic_ops_golang`, I just put everything > in the `main` package, with the assumption that we'd move compon

Re: Traffic Ops code re-structure

2018-11-14 Thread Robert Butts
Well, when I originally wrote `traffic_ops_golang`, I just put everything in the `main` package, with the assumption that we'd move components as they got big enough. It seems self-evident to me that these should be packages, very little should go in `main` in any nontrivial Go app. I don't really

Traffic Ops code re-structure

2018-11-14 Thread Fieck, Brennan
A couple of days ago I opened a pull request to add an API response to Traffic Ops's `/api/?$` endpoint, for the OPTIONS and GET HTTP methods. The form and function of those changes are still works-in-progress, and comments on them are welcome, but what I'm asking about today is more about the c