vscode-pull-request-github icon indicating copy to clipboard operation
vscode-pull-request-github copied to clipboard

'Not all changes could be converted to suggestions'

Open joshspicer opened this issue 1 year ago • 2 comments

Testing #6160

I may be understanding what "commenting range" here is, but I presumed that this change in the attached video should be ok?

https://github.com/user-attachments/assets/c4e1bef5-0c17-4e3c-8df1-c28f3e4cc692

joshspicer avatar Aug 28 '24 01:08 joshspicer

I don't any commenting ranges in the diff editor that you have open. Is that file part of the PR that y9ou currently have checked out?

alexr00 avatar Aug 28 '24 14:08 alexr00

Yes, it's part of a PR. In this example i've included the bottom status bar that shows this change is part of https://github.com/joshspicer/almost-empty/pull/6

I think i'm not understanding what constitutes a 'commenting range'? For example on line 4 here I made an edit (and commented on it), then ran the action from the SCM panel.

Image

joshspicer avatar Aug 29 '24 02:08 joshspicer

A commenting range is wherever you see the commenting bar and can leave a comment. If you make a change that's within that range then it should work, with the caveat that if you add or remove a line at the edge of a range it might not work because of where GitHub defines the ranges.

It looks like you made the change within a commenting range. Can you share a link to the PR by chance?

alexr00 avatar Aug 30 '24 13:08 alexr00

Here's the PR: https://github.com/joshspicer/almost-empty/pull/6

joshspicer avatar Aug 30 '24 22:08 joshspicer

The issue is when the only changes are line insertions.

alexr00 avatar Sep 02 '24 16:09 alexr00

I am no longer seeing the option available: image

joshspicer avatar Sep 03 '24 18:09 joshspicer

Verified with @joshspicer in a call.

alexr00 avatar Sep 03 '24 20:09 alexr00