search

zbirenbaum / copilot.lua

Fully featured & enhanced replacement for copilot.vim complete with API for interacting with Github Copilot
MIT License
2.43k stars 65 forks source link

Update copilot.vim/dist #202

Closed github-actions[bot] closed 11 months ago

github-actions[bot] commented 11 months ago

Automated changes by create-pull-request GitHub action

MunifTanjim commented 11 months ago

copilot server changed some output, this pr needs changes in lua codes.

zbirenbaum commented 11 months ago

Collaborator

I'll take care of this one then. I need to take note of the changes anyways since I'm currently standardizing responses from copilot-rs to match the distributed agent.

MunifTanjim commented 11 months ago

btw, I don't think they changed the shape of the response. Just slight change in the string content... because I see the string is processed a bit differently in https://github.com/github/copilot.vim/commit/97a9f1af97627b6aebeef864c981539ac1776b69

So just need to double check if everything is still working correctly in copilot.lua.

zbirenbaum commented 11 months ago

I think this is safe to merge. Suggestions don't seem to change behavior and there's no changes as far as I can tell in the responses.

Printing out a diff between this agent.js and an older one both formatted with prettier didn't reveal anything particularly notable either. As far as I can tell there's just one function that has a new parameter and some stuff relating to telemetry.

Maybe copilot.vim just was doing some cleanup or error handling?