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

bug: do not work on new commit to pull_request #32

Closed
Certseeds opened this issue Mar 2, 2023 · 3 comments
Closed

bug: do not work on new commit to pull_request #32

Certseeds opened this issue Mar 2, 2023 · 3 comments

Comments

@Certseeds
Copy link

refer to Certseeds/openpgpage#4
use actions to trigger code-review

name: Code Review

permissions:
  contents: read
  pull-requests: write

on:
  pull_request:
    types: [opened, reopened, synchronize]

jobs:
  test:
    if: github.repository == 'Certseeds/openpgpage'
    runs-on: ubuntu-latest
    steps:
      - uses: anc95/ChatGPT-CodeReview@v1
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          OPENAI_API_KEY: ${{ secrets.CHATGPT_SESSION_TOKEN }}

1st commit trigger a review, get a meaningful response.
2nd force-push trigger a review, find a meaingful response too.
but the 3rd and 4th push can trigger actions but do not find response, is that corect?
ps:
the 3rd action https://github.com/Certseeds/openpgpage/actions/runs/4315833869
the 4th action https://github.com/Certseeds/openpgpage/actions/runs/4315849686

@Certseeds
Copy link
Author

ps: maybe update action's json file with newest code is helpful

@anc95
Copy link
Owner

anc95 commented Mar 3, 2023

@Certseeds
Have you tried change the action to anc95/ChatGPT-CodeReview@main (the code on main branch) to see if it would work?

@Certseeds
Copy link
Author

@Certseeds Have you tried change the action to anc95/ChatGPT-CodeReview@main (the code on main branch) to see if it would work?

yes, it works.

the bug come from commit bb605aa

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