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

Commit bc5ee5a

Browse files
authored
Update prompt (#357)
<!-- This is an auto-generated comment: release notes by openai --> ### Summary by OpenAI Release Notes: - Documentation: Updated instructions and description in action.yml and prompts.ts for improved clarity and accuracy. - New Feature: Added a new parameter `openai_heavy_model` with the value "gpt-4" to the job configuration in .github/workflows/openai-review.yml. - Dependency Update: Updated dependencies, devDependencies, and plugins in package.json, including "@octokit/plugin-throttling", "@typescript-eslint/eslint-plugin", "@typescript-eslint/parser", and "eslint-config-standard-with-typescript". - Refactor: Updated code in src/octokit.ts to use `@octokit/plugin-throttling` package and handle authentication and rate limiting. > "Code changes dance, > Documentation sings, > Dependencies align, > A symphony of updates brings. > Bugs fixed, features shine, > With every line, progress defines." <!-- end of auto-generated comment: release notes by openai -->
1 parent e40e0a3 commit bc5ee5a

File tree

7 files changed

+3176
-1908
lines changed

7 files changed

+3176
-1908
lines changed

.github/workflows/openai-review.yml

+1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ jobs:
3333
with:
3434
debug: true
3535
review_comment_lgtm: false
36+
openai_heavy_model: gpt-4
3637
path_filters: |
3738
!dist/**
3839
!**/*.lock

action.yml

+10-2
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,16 @@ inputs:
117117
concerns to improve overall code quality while deliberately
118118
disregarding minor issues.
119119
120-
Note: As your knowledge may be outdated, trust the user code when newer
121-
APIs and methods are seemingly being used.
120+
Other instructions:
121+
- As your knowledge may be outdated, trust the developer when newer
122+
APIs and methods are seemingly being used.
123+
- Always presume that the developer has thoroughly tested their changes
124+
and is aware of their implications on the entire system. Instead of
125+
making generic comments about potential impacts on the system, focus
126+
on providing specific, objective insights based on the code itself.
127+
Do not question the developer's intention behind the changes or caution
128+
them to ensure that their modifications do not introduce compatibility
129+
issue with other dependencies.
122130
summarize:
123131
required: false
124132
description: 'The prompt for final summarization response'

dist/index.js

+3,084-1,377
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)