Skip to content
This repository has been archived by the owner on May 3, 2023. It is now read-only.

feature: Simplify usage #17

Open
keegancsmith opened this issue Aug 17, 2016 · 4 comments · May be fixed by #41 or #63
Open

feature: Simplify usage #17

keegancsmith opened this issue Aug 17, 2016 · 4 comments · May be fixed by #41 or #63
Labels
Est difficulty: 2 pts Request for commits!

Comments

@keegancsmith
Copy link
Member

Why don't we update Thyme CLI to have really smart defaults + "watch" built in. IE a user should just be able to run

$ thyme track
<ctrl-c>
$ thyme show
# opens thyme.html similiar to how go pprof opens webbrowser for svgs

and it would magically work.
@dufferzafar
Copy link

Going one step further, can't we just make all that transparent?

Just run thyme once, and it starts tracking in the background, dumping json etc.

It also opens up a local webserver to serve the output html file, so to view the current results, just goto localhost:6090 and they'll be served to you!

@HalfVoxel
Copy link

I don't think it is a good idea to start a web server without an explicit action from the user. But maybe a flag to be able to track and keep a server open at the same time would be a good idea.

Like

thyme --track --show

@seletskiy
Copy link
Contributor

@dufferzafar: No, it's part of the service manager job to daemonize tools, not tool by itself. E.g., using systemd, there should be .service file managing that.

@beyang
Copy link
Member

beyang commented Aug 18, 2016

I like this idea. Will happily accept a PR :)

@HalfVoxel HalfVoxel linked a pull request Aug 18, 2016 that will close this issue
Open
@Olreich Olreich linked a pull request Aug 19, 2016 that will close this issue
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Est difficulty: 2 pts Request for commits!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add watch command HalfVoxel/thyme
Add watch command Olreich/thyme
5 participants
@keegancsmith @seletskiy @HalfVoxel @dufferzafar @beyang