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

chore: code-review-quality.yml #3218

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

h1alexbel
Copy link
Contributor

@h1alexbel h1alexbel commented Jun 7, 2024

I've developed Code Review Quality GitHub Action: https://github.com/tracehubpm/code-review-action. In this PR I integrated workflow to the repository in order to enhance the quality of code reviews in it.

@maxonfjvipon take a look, please

PR-Codex overview

This PR adds a GitHub Actions workflow for code review quality using a custom action with OpenAI integration.

Detailed summary

  • Added MIT License header
  • Created code-review workflow
  • Triggers on submitted PR reviews
  • Executes on approval
  • Uses tracehub/code-review-action Docker image
  • Requires GitHub and OpenAI API tokens

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@maxonfjvipon
Copy link
Member

@h1alexbel sounds pretty interesting, could you please share some examples of how this tool works?

@h1alexbel
Copy link
Contributor Author

@maxonfjvipon sure.

image
image

Copy link
Member

@maxonfjvipon maxonfjvipon left a comment

Choose a reason for hiding this comment

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

@h1alexbel ok, let's try

@maxonfjvipon
Copy link
Member

@h1alexbel it seems something is wrong with your script

@h1alexbel
Copy link
Contributor Author

@maxonfjvipon should be fixed now. Could you re-approve this PR in order to re-run the workflow on a new version?

Copy link
Member

@maxonfjvipon maxonfjvipon left a comment

Choose a reason for hiding this comment

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

@h1alexbel ok, let's try

@maxonfjvipon
Copy link
Member

@h1alexbel it failed again :(

@h1alexbel
Copy link
Contributor Author

@maxonfjvipon released and tested new version, should work with valid credentials. If issue persists, there is a possibility that credentials are expired or invalid.

Copy link
Member

@maxonfjvipon maxonfjvipon left a comment

Choose a reason for hiding this comment

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

@h1alexbel ok, let's try again

@maxonfjvipon
Copy link
Member

@yegor256 could you please check if our GPT API keys are valid?

@h1alexbel
Copy link
Contributor Author

@maxonfjvipon based on logs for other OpenAI actions, looks like credentials are good. I will investigate what's wrong with my action and improve it

@maxonfjvipon
Copy link
Member

@h1alexbel what's up with this one? If it's in still progress - please make it a draft, or please close it otherwise

@h1alexbel h1alexbel marked this pull request as draft September 3, 2024 15:33
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

Successfully merging this pull request may close these issues.

2 participants