Skip to content
This repository was archived by the owner on Mar 6, 2024. It is now read-only.

Review comment context #50

Merged
merged 19 commits into from
Mar 18, 2023
Merged

Review comment context #50

Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9c61725
provide additional context
harjotgill Mar 18, 2023
f44b40c
test
harjotgill Mar 18, 2023
c46b2fa
refactor
harjotgill Mar 18, 2023
37aa9d0
refactor
harjotgill Mar 18, 2023
0b2a2ff
refactor
harjotgill Mar 18, 2023
0c75cb2
refactor
harjotgill Mar 18, 2023
92e97b0
refactor
harjotgill Mar 18, 2023
8f7d615
refactor
harjotgill Mar 18, 2023
1c229db
refactor
harjotgill Mar 18, 2023
0d42154
refactor
harjotgill Mar 18, 2023
8d7dc64
refactor
harjotgill Mar 18, 2023
449786f
refactor
harjotgill Mar 18, 2023
0603df1
refactor
harjotgill Mar 18, 2023
0e2a2e0
refactor
harjotgill Mar 18, 2023
9fbdda6
debug
harjotgill Mar 18, 2023
561b050
debug
harjotgill Mar 18, 2023
b0c8153
debug
harjotgill Mar 18, 2023
ce90c99
fix
harjotgill Mar 18, 2023
5f979ac
switch off debug
harjotgill Mar 18, 2023
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
2 changes: 1 addition & 1 deletion .github/workflows/openai-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,4 @@ jobs:
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
with:
debug: true
review_comment_lgtm: false
review_comment_lgtm: true
Copy link
Contributor

@github-actions github-actions bot Mar 18, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

🤖 OpenAI

This patch looks good to me. The change made is setting the review_comment_lgtm flag to true for debugging purposes, which seems reasonable and should not cause any issues.

LGTM!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will be making it false shortly.

Copy link
Contributor

Choose a reason for hiding this comment

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

🤖 OpenAI

@harjotgill: Thank you for your comment. It's great to see that you are actively managing the review_comment_lgtm flag and ensuring that it is set appropriately. Let us know if there is anything else we can help with.

Copy link
Contributor

@github-actions github-actions bot Mar 18, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

🤖 OpenAI

Based on the context and the diff, it seems that the change made is setting the review_comment_lgtm flag to true for debugging purposes. This seems reasonable and should not cause any issues.

LGTM!

8 changes: 7 additions & 1 deletion action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,11 @@ inputs:
default: |
$filename

Existing comments on the patch. Please use them as additional context.
```
$comment_chain
```

```diff
$patch
```
Expand Down Expand Up @@ -259,7 +264,8 @@ inputs:
```

Please reply directly to the new comment in the conversation
chain without extra prose as that reply will be posted as-is.
chain (instead of suggesting a reply) and your reply will be
posted as-is.

In your reply, please make sure to begin the reply by
tagging the user with "@user".
Expand Down
153 changes: 97 additions & 56 deletions dist/index.js
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading