camel icon indicating copy to clipboard operation
camel copied to clipboard

feat: unify timeout format with TIMEOUT constant

Open JINO-ROHIT opened this issue 1 month ago • 3 comments

This PR unifies the timeout threshold using a Constant across all the files.

JINO-ROHIT avatar Nov 11 '25 07:11 JINO-ROHIT

[!IMPORTANT]

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment
  • [ ] Commit unit tests in branch unify-timeout

[!TIP]

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example:

"Create a concise high-level summary as a bullet-point list. Then include a Markdown table showing lines added and removed by each contributing author."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot] avatar Nov 11 '25 07:11 coderabbitai[bot]

@MuggleJinx for subclasses, should i default to none or remove the timeout param since it inherits from base already?

JINO-ROHIT avatar Nov 13 '25 18:11 JINO-ROHIT

@MuggleJinx for subclasses, should i default to none or remove the timeout param since it inherits from base already?

yeah I think so, you can just define the default value in base class.

MuggleJinx avatar Nov 14 '25 00:11 MuggleJinx

i agree, i mustve missed this one, done now!

JINO-ROHIT avatar Nov 17 '25 15:11 JINO-ROHIT