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

feat: add openai-pr-reviewer #4424

Merged

Conversation

yukkysaito
Copy link
Contributor

@yukkysaito yukkysaito commented Jul 27, 2023

Description

🤖 Generated by Copilot at 71efae0

Add a GitHub workflow to use OpenAI Codex for PR reviews. The workflow file .github/workflows/openai-pr-reviewer.yaml invokes the fluxninja/openai-pr-reviewer action with some configuration options.

Related links

Tests performed

Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.
  • The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>
@github-actions github-actions bot added the type:ci Continuous Integration (CI) processes and testing. (auto-assigned) label Jul 27, 2023
@yukkysaito
Copy link
Contributor Author

Testing 🙏

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>
Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>
Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>
Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>
@yukkysaito yukkysaito requested a review from mitsudome-r July 27, 2023 15:11
Copy link
Member

@mitsudome-r mitsudome-r left a comment

Choose a reason for hiding this comment

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

LGTM

@mitsudome-r mitsudome-r added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Jul 27, 2023
@yukkysaito
Copy link
Contributor Author

The Action test does not work well in the forked repository, but we were able to confirm that it works well in this PR.

@yukkysaito yukkysaito merged commit 19dc46e into autowarefoundation:main Jul 27, 2023
soblin pushed a commit to soblin/autoware.universe that referenced this pull request Jul 29, 2023
* feat: add openai-pr-reviewer

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>

* add OPENAI_API_ORG

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>

* change openai_heavy_model

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>

* remove OPENAI_API_ORG

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>

* Change action name

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>

---------

Signed-off-by: Yukihito Saito <yukky.saito@gmail.com>
Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) type:ci Continuous Integration (CI) processes and testing. (auto-assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants