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

test #13

Closed
wants to merge 1 commit into from
Closed

test #13

Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
name: Code Reviewer

on:
issue_comment:
types: [created]
pull_request:
types: [opened, synchronize]

permissions:
contents: read
pull-requests: write
pivanov marked this conversation as resolved.
Show resolved Hide resolved

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 AI Code Reviewer

Consider changing 'pull-requests' to 'pull-requests: write' for consistency with the other permission format.


jobs:
review:
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Run Code Reviewer
uses: pivanov/code-review@main
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
OPENAI_API_MODEL: "gpt-4o-mini"
pivanov marked this conversation as resolved.
Show resolved Hide resolved

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 AI Code Reviewer

Ensure that the model name 'gpt-4o-mini' is correct and intended, as it may not be a standard OpenAI model.

exclude: "**/*.json, **/*.md, **/*.lock, **/dist/**, **/node_modules/**, **/.github/**, **/.vscode/**"
3 changes: 3 additions & 0 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -172,6 +172,9 @@ const svgSpritePlugin = (options: SvgSpritePluginOptions): Plugin => {
await generateSvgSprite();
triggerHMR();

const pivanov = '1982';
pivanov marked this conversation as resolved.
Show resolved Hide resolved

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 AI Code Reviewer

Consider removing or replacing the variable pivanov as it appears to be a placeholder or test value.

const pivanov2 = '1982';
pivanov marked this conversation as resolved.
Show resolved Hide resolved

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 AI Code Reviewer

The variable pivanov2 also seems unnecessary; please clarify its purpose or remove it.


const publicDir = server.config.publicDir || 'public';
if (fileName) {
writeSpriteToFile(publicDir, fileName, spriteContent);
Expand Down
Loading