-
Notifications
You must be signed in to change notification settings - Fork 262
Conversation
🤖 OpenAI
Chat with 🤖 OpenAI Bot (
|
if (summarizeResp === '') { | ||
info('summarize: nothing obtained from openai') | ||
summariesFailed.push(`${filename} (nothing obtained from openai)`) | ||
info('summarize: nothing fetched from RedRover') | ||
summariesFailed.push(`${filename} (nothing fetched from RedRover)`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
Replace the log message to maintain consistency with other log messages.
if (summarizeResp === '') { | |
info('summarize: nothing obtained from openai') | |
summariesFailed.push(`${filename} (nothing obtained from openai)`) | |
info('summarize: nothing fetched from RedRover') | |
summariesFailed.push(`${filename} (nothing fetched from RedRover)`) | |
info('summarize: nothing fetched from RedRover for ' + filename) | |
summariesFailed.push(`${filename} (nothing fetched from RedRover)`) |
} catch (e: any) { | ||
warning(`summarize: error from openai: ${e as string}`) | ||
summariesFailed.push(`${filename} (error from openai: ${e as string})})`) | ||
warning(`summarize: error from RedRover: ${e as string}`) | ||
summariesFailed.push(`${filename} (error from RedRover: ${e as string})})`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
Maintain consistency in log messages and error handling.
} catch (e: any) { | |
warning(`summarize: error from openai: ${e as string}`) | |
summariesFailed.push(`${filename} (error from openai: ${e as string})})`) | |
warning(`summarize: error from RedRover: ${e as string}`) | |
summariesFailed.push(`${filename} (error from RedRover: ${e as string})})`) | |
warning(`summarize: error from RedRover for ${filename}: ${e as string}`) | |
summariesFailed.push(`${filename} (error from RedRover: ${e as string})`) |
if (summarizeResp === '') { | ||
warning('summarize: nothing obtained from openai') | ||
warning('summarize: nothing fetched from RedRover') | ||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
Update the log message for consistency.
if (summarizeResp === '') { | |
warning('summarize: nothing obtained from openai') | |
warning('summarize: nothing fetched from RedRover') | |
} else { | |
warning('summarize: nothing fetched from RedRover for final response') |
if (summarizeFinalResponse === '') { | ||
info('summarize: nothing obtained from openai') | ||
info('summarize: nothing fetched from RedRover') | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
Update the log message for consistency.
if (summarizeFinalResponse === '') { | |
info('summarize: nothing obtained from openai') | |
info('summarize: nothing fetched from RedRover') | |
} | |
info('summarize: nothing fetched from RedRover for final response') |
if (releaseNotesResponse === '') { | ||
info('release notes: nothing obtained from openai') | ||
info('release notes: nothing fetched from RedRover') | ||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
Update the log message for consistency.
if (releaseNotesResponse === '') { | |
info('release notes: nothing obtained from openai') | |
info('release notes: nothing fetched from RedRover') | |
} else { | |
info('release notes: nothing fetched from RedRover') |
if (response === '') { | ||
info('review: nothing obtained from openai') | ||
info('review: nothing fetched from RedRover') | ||
reviewsFailed.push(`${filename} (no response)`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
Update the log message for consistency.
if (response === '') { | |
info('review: nothing obtained from openai') | |
info('review: nothing fetched from RedRover') | |
reviewsFailed.push(`${filename} (no response)`) | |
info('review: nothing fetched from RedRover for ' + filename) | |
reviewsFailed.push(`${filename} (nothing fetched from RedRover)`) |
{} | ||
) | ||
if (releaseNotesResponse === '') { | ||
info('release notes: nothing obtained from openai') | ||
info('release notes: nothing fetched from RedRover') | ||
} else { | ||
let message = '### Summary by OpenAI\n\n' | ||
let message = '### Summary by RedRover\n\n' | ||
message += releaseNotesResponse | ||
try { | ||
await commenter.updateDescription( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 OpenAI
In line 380, change the summary message to include "RedRover" instead of "OpenAI".
{} | |
) | |
if (releaseNotesResponse === '') { | |
info('release notes: nothing obtained from openai') | |
info('release notes: nothing fetched from RedRover') | |
} else { | |
let message = '### Summary by OpenAI\n\n' | |
let message = '### Summary by RedRover\n\n' | |
message += releaseNotesResponse | |
try { | |
await commenter.updateDescription( | |
let message = '### Summary by RedRover\n\n' | |
message += releaseNotesResponse | |
try { | |
await commenter.updateDescription( |
Summary by OpenAI
New Feature:
Refactor: