nebraska icon indicating copy to clipboard operation
nebraska copied to clipboard

reorganize `cmd/nebraska/`

Open vbatts opened this issue 3 years ago • 1 comments

there is way too much logic in there, along with the main().

  • move out the controller to a ./pkg/...
  • move out the router setup to a ./pkg/...
    • possibly splitting out the groups of routes in to functions like API, Omaha, etc
  • get the ./cmd/nebraska/ down to a thin veil of parsing flags and called Execute

This will make testing nicer and easier, also will make it possible for clients to import and use shared router for calling the API

vbatts avatar Apr 14 '21 18:04 vbatts

Hey, I would love to take up this issue and work on it.

P.S.: I am new to Nebraska project. I worked on a good first issue before and now I am keen on working on this issue. Do you think I can get this up?

abraarsyed avatar Jul 09 '21 11:07 abraarsyed