Code review #11
Code review #11
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -0,0 +1,27 @@ | |||
| name: Code Reviewer | |||
There was a problem hiding this comment.
O nome do fluxo de trabalho não está claro. Considere renomeá-lo para algo mais descritivo.
| @@ -0,0 +1,27 @@ | |||
| name: Code Reviewer | |||
| run-name: Action started by ${{ github.actor }} | |||
There was a problem hiding this comment.
O nome do trabalho não está claro. Considere renomeá-lo para algo mais descritivo.
| - opened | ||
| - synchronize | ||
|
|
||
| permissions: write-all |
There was a problem hiding this comment.
As permissões 'write-all' são muito amplas. Considere restringir as permissões necessárias para este trabalho.
| @@ -0,0 +1,243 @@ | |||
| require('child_process') | |||
There was a problem hiding this comment.
Importação desnecessária: 'child_process'.
| @@ -0,0 +1,243 @@ | |||
| require('child_process') | |||
| .execSync( | |||
| 'npm install @actions/core @actions/github parse-diff minimatch fs', | |||
There was a problem hiding this comment.
Comando 'npm install' sendo executado de forma síncrona.
| name: emanuel-braz/code-review | ||
| description: "Perform code reviews and comment on diffs using OpenAI API." | ||
| author: Emanuel Braz | ||
| branding: |
There was a problem hiding this comment.
A propriedade 'branding' não é necessária para o funcionamento do código.
| description: "OpenAI API model." | ||
| required: false | ||
| default: "gpt-3.5-turbo" | ||
| exclude: |
There was a problem hiding this comment.
A propriedade 'exclude' não é necessária para o funcionamento do código.
| @@ -29,6 +29,18 @@ class SimpleChatGptService { | |||
| throw error; | |||
| @@ -29,6 +29,18 @@ class SimpleChatGptService { | |||
| throw error; | |||
| } | |||
| } | |||
|
|
|||
| // chatCompletionParams: ChatCompletionParams | |||
There was a problem hiding this comment.
Adicione um comentário descrevendo a função 'fromParams'.
| console.error('[SimpleChatGptService]', error); | ||
| throw error; | ||
| } | ||
| } | ||
| } | ||
|
|
||
| module.exports = SimpleChatGptService; |
There was a problem hiding this comment.
Adicione uma nova linha no final do arquivo.
No description provided.