coderabbitai / ai-pr-reviewer

AI-based Pull Request Summarizer and Reviewer with Chat Capabilities.
https://coderabbit.ai
MIT License
1.42k stars 261 forks source link

do not retry on secondary limits #380

Closed harjotgill closed 11 months ago

harjotgill commented 11 months ago

Summary by OpenAI

Refactor:

🎉 Here's to the code that's leaner, not meaner, To the logs that are clearer, making debugging no fearer. With each retry counted, and rate limits mounted, We've refactored and pruned, and a victory is sounded! 🥳

github-actions[bot] commented 11 months ago

:robot: OpenAI

High-Level Summary

The changes primarily focus on modifying the logging mechanism in src/octokit.ts. The warning import is replaced with a standard console.log statement. Additionally, a condition is introduced to check the retry count before logging and returning true. The function onSecondaryRateLimit now uses this updated logging approach. Lastly, the retry configuration object has been removed.

File Summary
src/octokit.ts Replaced warning import with console.log, added retry count check before logging, updated onSecondaryRateLimit function to use new logging method, and removed retry configuration object.

Chat with 🤖 OpenAI Bot (@openai)

Code suggestions

Ignoring further reviews



In the recent run, only the files that changed from the base of the PR and between 955c447192059dd08421ff2640cbe195a156f7fe and 74dd0c1f3d5fc1f5916fa3d0a46fc7f4854d5b99 commits were reviewed.