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

[feature] Show creator of a todo #1480

Closed
anbraten opened this issue Apr 15, 2024 · 3 comments · Fixed by #1515
Closed

[feature] Show creator of a todo #1480

anbraten opened this issue Apr 15, 2024 · 3 comments · Fixed by #1515
Assignees
Labels
enhancement New feature or request

Comments

@anbraten
Copy link

Is your feature request related to a problem? Please describe.
In larger projects it would be quite nice to see who added a todo, to maybe be able to later on filter on my own todos.

Describe the solution you'd like
In case no username was detected / or maybe always check which git commit added the specific line initially which would the creator / person who cares for it in most cases.

Describe alternatives you've considered
I could simply add my user to all todo comments. This would however prevent me from adding a #issue,

@anbraten anbraten added enhancement New feature or request triage This issue needs triage labels Apr 15, 2024
@ianlewis
Copy link
Owner

@anbraten Where would you expect the author of the todo to come from?

@anbraten
Copy link
Author

Sth like git blame -w --porcelain --incremental -L 23,23 server/store/store.go.

I got this idea from tickgit which is using the git blame command in the code.

@ianlewis ianlewis removed the triage This issue needs triage label May 8, 2024
ianlewis added a commit that referenced this issue Jul 13, 2024
Add a new `--blame` option which tells `todos` to try and get the VCS
committer of each TODO. Currently adds significant slowdown.

Fixes #1480

Signed-off-by: Ian Lewis <ianmlewis@gmail.com>
@ianlewis ianlewis mentioned this issue Jul 13, 2024
7 tasks
@ianlewis ianlewis linked a pull request Jul 13, 2024 that will close this issue
7 tasks
@anbraten
Copy link
Author

Thanks for adding this

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

Successfully merging a pull request may close this issue.

2 participants