-
Notifications
You must be signed in to change notification settings - Fork 626
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
Author date / commit date confusion #294
Comments
It will probably take some considerable work to upgrade all views to support both commit and author dates. In the mean time the docs should be updated to use "author date" instead of "commit date". |
Thanks for looking into it. I feared as much by poking around the code. I love tig so thanks for this wonderful tool! (And if there's a quick hack you can describe for a personal fork that always uses the commit date instead of the author date, I'm happy to hack and use it. Otherwise, feel free to close the issue.) |
This changes two 'commit date' to 'author date'. References #294
Looks like there's an open PR to address this: #810 |
It would be great to toggle between the author and commit date. Especially in the |
To being able to see commit date would be extremely useful. As in most projects author date is almost an arbitrary thing, but committer date really shows when change is introduced. |
@koutcher Thanks much! |
@koutcher Awesome, thanks a lot! |
7351f17 has some issues
For some projects, author fields are useful for 3rd party/picked changes. I closed #810 as it was too old, but I really want the feature for name+date. |
The main view displays the "Author Date" (though all the documentation for
tig
andtigrc
refers to it as the "Commit Date".) I would really like to see the actual Commit Date. I assume that it isn't supported (I didn't see an option for it – I just built a fresh tig from source.) The Commit Date is super helpful when scanning the upstream log to see which breaking commits are newer and and require rebase attention. (I missed a conflicting commit because I thought I had already rebased on it.) I would love to see some support this.The text was updated successfully, but these errors were encountered: