Skip to content

Conversation

@algmyr
Copy link
Member

@algmyr algmyr commented Nov 30, 2025

Will allow things like showing info about conflicted files in the log.

Should synergize well with #8184

Fixes: #7377

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

@algmyr algmyr force-pushed the algmyr/push-nokmyqrvmwmv branch from 63dc14a to f65bf08 Compare November 30, 2025 16:47
@algmyr algmyr marked this pull request as ready for review November 30, 2025 16:47
@algmyr algmyr requested a review from a team as a code owner November 30, 2025 16:47
@algmyr algmyr force-pushed the algmyr/push-nokmyqrvmwmv branch 2 times, most recently from 93e1a64 to 2662450 Compare December 2, 2025 00:31
@algmyr algmyr force-pushed the algmyr/push-nokmyqrvmwmv branch from 2662450 to c261ca9 Compare December 2, 2025 00:39
},
);
map.insert(
"num_conflict_sides",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I slightly prefer conflict_sides or conflict_side_count, but I don't feel strongly.

Another idea is to add added/removed_entries() -> List<TreeEntry>.


// Merge with conflicts.
work_dir
.run_jj(["new", "description(left)", "description(right)"])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: use subject(glob:left) (it's equivalent to exact:left, but I'll remove glob: when it becomes the default.)

let output = work_dir.run_jj([
"log",
"-r",
"description(substring:\"left\")",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: same here. subject(glob:left)

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.

FR: Template: List conflicted files in a Commit

4 participants