pivotal/skenario

Remove CLI trace

jchesterpivotal opened this issue · 1 comments

jchesterpivotal commented

Currently cmd/skenario is the CLI with trace output. It's pretty and I like showing it off, but realistically the bulk of the value is in the visualisation produced by cmd/serve. The business of studying traces can be achieved more directly using sqlite3 to query skenario.db.

Moreover, having two code paths increases the testing and documentation burden.

The proposal here is to:

  1. Remove cmd/skenario
  2. Rename cmd/serve to cmd/skenario.

@josephburnett any objection?

jchesterpivotal commented

There being no objections, I'm going to go ahead with this.