Skip to content

create a bot test script - #1

Open
wenboown wants to merge 2 commits into
mainfrom
action-test
Open

create a bot test script#1
wenboown wants to merge 2 commits into
mainfrom
action-test

Conversation

@wenboown

@wenboown wenboown commented May 7, 2024

Copy link
Copy Markdown

No description provided.

@wenboown

wenboown commented May 7, 2024

Copy link
Copy Markdown
Author

@SeineSailor test run

@wenboown

wenboown commented May 7, 2024

Copy link
Copy Markdown
Author

the last run verified:

  1. the @ feature works. the action only runs when called.

But the options doesn't seem to be set correctly.

openai_light_model: 
openai_heavy_model: 
openai_model_temperature: NaN
openai_retries: NaN
openai_timeout_ms: NaN
openai_concurrency_limit: NaN
github_concurrency_limit: 6
summary_token_limits: max_tokens=4000, request_tokens=2900, response_tokens=1000
review_token_limits: max_tokens=4000, request_tokens=2900, response_tokens=1000
api_base_url: 
language: 

It shouldn't print out openai anymore?

@SeineSailor test again

@wenboown

wenboown commented May 7, 2024

Copy link
Copy Markdown
Author

@SeineSailor test again.

I deleted the with from the openai-review.yml

@wenboown

wenboown commented May 7, 2024

Copy link
Copy Markdown
Author

need to rebuild the dist folder for new code to work.

@SeineSailor run again

@wenboown

wenboown commented May 7, 2024

Copy link
Copy Markdown
Author

@SeineSailor use source map to help debug

@wenboown

wenboown commented May 7, 2024

Copy link
Copy Markdown
Author

editing comments does trigger the run. Now run again with new comments:

@SeineSailor use source map to help debug

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.

1 participant