Skip to content

Conversation

pabl0
Copy link
Contributor

@pabl0 pabl0 commented Mar 22, 2025

  • gptel-rewrite.el (gptel--rewrite-diff): Show the difference between buffers rather than file names. Otherwise, if a) the buffer has never been saved, the error "No such file or directory" will occur, or b) if there are unsaved changes in the buffer, the diff output will not reflect those changes.

* gptel-rewrite.el (gptel--rewrite-diff): Show the difference
between buffers rather than file names.  Otherwise, if a) the
buffer has never been saved, the error "No such file or
directory" will occur, or b) if there are unsaved changes in
the buffer, the diff output will not reflect those changes.
@karthink karthink merged commit 0ce917a into karthink:master Mar 22, 2025
@karthink
Copy link
Owner

Thanks @pabl0! I don't remember why I set it up to look for a file -- it was definitely deliberate considering the trouble I went to there instead of just calling diff-no-select the usual way. If something breaks I guess we'll find out.

aagit added a commit to aagit/gptel that referenced this pull request Oct 9, 2025
This reverts commit 0ce917a.

The above commit while not wrong per se, prevents to use diff-mode to
apply hunks, which makes it not a good tradeoff.

Signed-off-by: Andrea Arcangeli <[email protected]>
aagit added a commit to aagit/gptel that referenced this pull request Oct 9, 2025
This reverts commit 0ce917a.

The above commit while not wrong per se, prevents to use diff-mode to
apply hunks, which makes it not a good tradeoff.
aagit added a commit to aagit/gptel that referenced this pull request Oct 9, 2025
This reverts commit 0ce917a.

The above commit while not wrong per se, prevents to use diff-mode to
apply hunks, which makes it not a good tradeoff.
aagit added a commit to aagit/gptel that referenced this pull request Oct 15, 2025
This reverts commit 0ce917a.

The above commit while not wrong per se, prevents to use diff-mode to
apply hunks, which makes it not a good tradeoff.
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