Skip to content

Conversation

avamsi
Copy link
Member

@avamsi avamsi commented Oct 8, 2025

Would be easier to type when using with other commands -- of course, one
could always use an alias too, especially if -o is too confusing.

Checklist

If applicable:

  • I have updated CHANGELOG.md
  • I have updated the documentation (README.md, docs/, demos/)
  • I have updated the config schema (cli/src/config-schema.json)
  • I have added/updated tests to cover my changes

@avamsi avamsi requested a review from a team as a code owner October 8, 2025 18:16
Change-Id: Id0000000bfbf7d9493e9ae79600c53bb0bf62376
@avamsi avamsi force-pushed the av/jj_okoksmqvqwlq branch from 0353100 to ae6d449 Compare October 8, 2025 18:57
@yuja
Copy link
Contributor

yuja commented Oct 9, 2025

Do you want both jj log -o and jj diff -o? If the latter is what you use often, I think user alias is better. (FWIW, I tend to use --stat or --summary with jj log.)

@avamsi
Copy link
Member Author

avamsi commented Oct 9, 2025

Yeah, I've never used jj log --name-only (didn't even realize it existed 😅) and I don't think I will (definitely not often enough to merit a shorthand). I could make the change just to be consistent, or just continue using an alias for jj diff --name-only (I'm leaving the PR open in case anyone's interested, but @yuja / other maintainers, please feel free to close).


jj log -s is definitely interesting, I use this script to do something similar:

print '@ Working copy changes:'
# We intentionally don't use bg(c) here to let jj automatically commit
# changes to the working copy (so it's not stale on output).
jj --color=always diff --summary --revision=@ | indent
conflicts=$(jj bgc resolve --list 2>/dev/null) && {
	print '\n\e[31mx Conflicts:\e[0m'
	print "$conflicts" | indent
}
print '\n| Parent commit changes:'
jj bgc diff --summary --revision='parents(@)' | indent
print
jj bgc log --revisions=interesting
[aliases]
        bg = [
                '--config=ui.log-word-wrap=false',
                '--ignore-working-copy',
                '--no-pager']
        bgc = ['bg', '--color=always']

Perhaps it's possible to simplify it now, I'll look at the changelog, thank you!

@avamsi avamsi closed this Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants