🚀 Configurable timestamp output format in git blame

[mliszcz]

In git blame I noticed this:

It would be helpful to know e.g. which change was applied first but this can not be figured out from the humanized timestamp.

delta/src/handlers/blame.rs

Line 273 in 738c5a1

chrono_humanize::HumanTime::from(blame.time).to_string(),

I changed it locally to: blame.time.format("%H:%M %Y-%m-%d").to_string(),.

Would it be possible to make this configurable? Perhaps a new option --blame-output-timestamp-format could be added. If present (and not empty) it would be used as a format string. If not, we will fall back to HumanTime formatting. What do you think?

I can work on a PR if this could be accepted.

[dandavison]

Hi @mliszcz, thanks! I agree with the feature proposal and I think that your proposed API is spot-on. Please do go ahead and make a PR if you have time, and let me know if anything's unclear about the development workflow.

[mliszcz]

Thanks. I've just opened a PR. Please review carefully because I'm doing rust for the first time! I guess the formatter job will most likely fail, I will run that tomorrow and eventually amend the commit.

[mliszcz]

Ready. @dandavison please have a look and trigger the pipeline when you find some time.