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

Commit

Permalink
Update prompt (#357)
Browse files Browse the repository at this point in the history
<!-- 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 -->
  • Loading branch information
harjotgill authored Jul 14, 2023
1 parent e40e0a3 commit bc5ee5a
Show file tree
Hide file tree
Showing 7 changed files with 3,176 additions and 1,908 deletions.
1 change: 1 addition & 0 deletions .github/workflows/openai-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ jobs:
with:
debug: true
review_comment_lgtm: false
openai_heavy_model: gpt-4
path_filters: |
!dist/**
!**/*.lock
12 changes: 10 additions & 2 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,8 +117,16 @@ inputs:
concerns to improve overall code quality while deliberately
disregarding minor issues.
Note: As your knowledge may be outdated, trust the user code when newer
APIs and methods are seemingly being used.
Other instructions:
- As your knowledge may be outdated, trust the developer when newer
APIs and methods are seemingly being used.
- Always presume that the developer has thoroughly tested their changes
and is aware of their implications on the entire system. Instead of
making generic comments about potential impacts on the system, focus
on providing specific, objective insights based on the code itself.
Do not question the developer's intention behind the changes or caution
them to ensure that their modifications do not introduce compatibility
issue with other dependencies.
summarize:
required: false
description: 'The prompt for final summarization response'
Expand Down
4,461 changes: 3,084 additions & 1,377 deletions dist/index.js

Large diffs are not rendered by default.

Loading

0 comments on commit bc5ee5a

Please sign in to comment.