unisonweb / unison

A friendly programming language from the future
https://unison-lang.org
Other
5.81k stars 271 forks source link

feat: render merge conflicts preferring Alice's names #5408

Closed mitchellwrosen closed 1 month ago

mitchellwrosen commented 1 month ago

Overview

Fixes #5208

This PR adjusts the PPE we use for rendering merge conflicts to prefer Alice's names when available (even for Bob's stuff). Previously, we'd render Alice's stuff preferring her names, and Bob's stuff preferring his names.

This makes merges easier to resolve as they will no longer have spurious differences in mergetools (or when merging manually) that could occur whenever Alice and Bob have different sets of aliases for the same term.

Test coverage

I've added a transcript that demonstrates the change