Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unhandled Error when file is deleted #43

Open
varun5001 opened this issue Oct 4, 2023 · 1 comment
Open

Unhandled Error when file is deleted #43

varun5001 opened this issue Oct 4, 2023 · 1 comment

Comments

@varun5001
Copy link

varun5001 commented Oct 4, 2023

GPTPRReviewer task fails when file is deleted from Repo and a PR is submitted. Below is Run Log of the task

`Changed Files (excluding binary files) :
test.py
Start deleting existing comments added by the previous Job ...
Existing comments deleted.
Start reviewing test.py with openai ...
##[error]Unhandled: fatal: ambiguous argument 'test.py': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git [...] -- [...]'

##[error]Error: fatal: ambiguous argument 'test.py': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git [...] -- [...]'

at Object.action (/home/vsts/work/_tasks/GPTPRReviewer_54bfc437-996c-4844-9f68-cd33e7a56682/0.6.58/node_modules/simple-git/dist/cjs/index.js:1274:25)
at PluginStore.exec (/home/vsts/work/_tasks/GPTPRReviewer_54bfc437-996c-4844-9f68-cd33e7a56682/0.6.58/node_modules/simple-git/dist/cjs/index.js:1309:29)
at Promise (/home/vsts/work/_tasks/GPTPRReviewer_54bfc437-996c-4844-9f68-cd33e7a56682/0.6.58/node_modules/simple-git/dist/cjs/index.js:1676:43)
at new Promise (<anonymous>)
at GitExecutorChain.handleTaskData (/home/vsts/work/_tasks/GPTPRReviewer_54bfc437-996c-4844-9f68-cd33e7a56682/0.6.58/node_modules/simple-git/dist/cjs/index.js:1674:16)
at GitExecutorChain.<anonymous> (/home/vsts/work/_tasks/GPTPRReviewer_54bfc437-996c-4844-9f68-cd33e7a56682/0.6.58/node_modules/simple-git/dist/cjs/index.js:1658:44)
at Generator.next (<anonymous>)
at fulfilled (/home/vsts/work/_tasks/GPTPRReviewer_54bfc437-996c-4844-9f68-cd33e7a56682/0.6.58/node_modules/simple-git/dist/cjs/index.js:55:24)
at process._tickCallback (internal/process/next_tick.js:68:7)

`

@Jura
Copy link

Jura commented Oct 26, 2023

Same here, review always failing when PR contains deleted files. Error stack is identical to the one above.

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

No branches or pull requests

2 participants