Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Idea: Add Starship integration/Status #109

Closed
sfrique opened this issue Jun 8, 2021 · 6 comments
Closed

Idea: Add Starship integration/Status #109

sfrique opened this issue Jun 8, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@sfrique
Copy link
Contributor

sfrique commented Jun 8, 2021

Hello folks,

I used watson tool from time to time, but it's python and it keeps breaking because of some dependencies from time to time.

Looking for other tools I found timetrace and others like rtw and timetracking

timtrace seems very active and I have like golang over rust, also this seems to have a better community.

So I am going to start to use timetrace

With all that said, I would like to see if we can add starship integration, it should be simple enough like see here: https://github.com/hardliner66/timetracking#starship and https://github.com/PicoJr/rtw#starship-prompt-integration
full doc on custom commands for starship: https://starship.rs/config/#custom-commands

I can try to implement it.

I would also like if people have Ideas on what output should we allow.
Like, enable arbitrary format like the examples on the tools above:
--format "{h}h {mm}m"
--format "{ongoing} {human_duration}"

I think initially what it would be nice is to get the total amount worked today as the current status show this information already.

But at some point, I would like to be able to show only worked time for the project on that day and so on, but this may be added to the report command and not the status.

Please let me know what you guys think.

@dominikbraun
Copy link
Owner

Thanks for filing this issue! This seems to be the same requirement as #94.

@dominikbraun dominikbraun added the enhancement New feature or request label Jun 8, 2021
@dominikbraun
Copy link
Owner

I assigned you in case you want to implement this feature. Let me know if you don't have the time capacities for implementing it 😄

@sfrique
Copy link
Contributor Author

sfrique commented Jun 8, 2021

Thanks for filing this issue! This seems to be the same requirement as #94.

I found that issue but forgot to add it to the description.

The purpose seems to be the same, but the implementation is different.

I can also try to add json as an output, but I think would be best to implement it in a separated pull request right?

@dominikbraun
Copy link
Owner

dominikbraun commented Jun 8, 2021

In my opinion, it would make sense to add a --output (short -o flag) that accepts json as a value to solve #94. This is also the way records will be displayed as JSON/CSV. To solve this issue however, an additional --format flag would make sense.

@sfrique
Copy link
Contributor Author

sfrique commented Jun 12, 2021

The basic implementation is merged: #114

@sfrique sfrique closed this as completed Jun 12, 2021
@dominikbraun
Copy link
Owner

@sfrique I opened a PR that documents the Starship integration: #124

Feel free to review it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants