fix: Check for Jujutsu repo at conflict location #462

Merged
wetneb merged 1 commit from senekor-kwlupwtkynpr into main 2025-07-02 16:16:13 +02:00
Member

We warn Jujutsu users when they are using the command solve, because
that likely won't work. This is a problem for contributors who use
Jujutsu. For example, tests of the solve command might fail, because
Jujutsu is detected.

To prevent this, we check if there is a Jujutsu repository at the
location of the conflicted file that was passed to the solve command
(instead of the current working directory). Our tests run in temporary
directories, where no Jujutsu repository will be detected.

We warn Jujutsu users when they are using the command `solve`, because that likely won't work. This is a problem for contributors who use Jujutsu. For example, tests of the `solve` command might fail, because Jujutsu is detected. To prevent this, we check if there is a Jujutsu repository at the location of the conflicted file that was passed to the `solve` command (instead of the current working directory). Our tests run in temporary directories, where no Jujutsu repository will be detected.
senekor force-pushed senekor-kwlupwtkynpr from 819cf3ce7b to 2fb867bcb7 2025-07-02 16:07:16 +02:00 Compare
ada4a approved these changes 2025-07-02 16:12:31 +02:00
ada4a left a comment
Owner

makes sense!

makes sense!
wetneb merged commit cfb0ca9cdd into main 2025-07-02 16:16:13 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mergiraf/mergiraf#462
No description provided.