-
Notifications
You must be signed in to change notification settings - Fork 365
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
templates: respect format_short_signature() in builtin_log_oneline #4714
base: main
Are you sure you want to change the base?
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
@ilyagr I just noticed that you also want to have |
I don't have a strong opinion, but whether or not we decide to show the author in the default annotation template (as discussed in #4653), Emily's idea of a For example, in #4653 (comment), I was thinking that Another possible value for In fact, one could argue that this template already effectively exists, it's just unnamed and present in only one place. I personally like formalizing I don't use the oneline template enough to have a strong opinion. |
I’m in the same boat as @emilazy:
So, it’s not the default I particularly care about — just the ability to customize the name display, without having to copy templates into my config and keep up with future jj changes. |
In that case, it sounds like creating a "very short signature" template alias will help you now, will work for a future annotation template, and will be a noop for everybody else. That sounds good to me, even if we later decide to change our mind and, say, unify two of the signature aliases or change the default "very short" template. So, let's do that unless we hear another opinion? |
At this rate, perhaps we could simply hard‐code it to output |
Just to be clear, I'm waiting for you to adjust the PR and I'm assuming you just haven't gotten around to it yet (which is totally fine!). But let me know if you're waiting on something instead. |
Yeah, just haven’t gotten around to it; I’ll convert it into draft for now |
I noticed that
jj log -T builtin_log_oneline
wasn't respecting myformat_short_signature()
settings.This is a slightly breaking change -- people's
oneline
logs will become longer if they use the defaultformat_short_signature()
. Perhaps there should be a new override, eg.format_tiny_signature()
?Checklist
not updating tests/changelog yet since idk if the PR will be merged
If applicable:
CHANGELOG.md