Skip to content

Conversation

himanshusinghs
Copy link
Collaborator

Proposed changes

In the last PR #536, I forgot to update the README with the description of the new config variables. This PR addressed the same.

Checklist

@himanshusinghs himanshusinghs requested a review from a team as a code owner September 22, 2025 09:20
@Copilot Copilot AI review requested due to automatic review settings September 22, 2025 09:20
@himanshusinghs himanshusinghs added the no-title-validation Add this label to disable the title check for this PR. label Sep 22, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates README to document newly introduced configuration variables and clarify limits for find and aggregate tools.

  • Added detailed descriptions for find and aggregate tool output size and document count limits.
  • Added documentation for maxBytesPerQuery and maxDocumentsPerQuery configuration options.

@coveralls
Copy link
Collaborator

coveralls commented Sep 22, 2025

Pull Request Test Coverage Report for Build 17910792258

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.06%) to 82.391%

Totals Coverage Status
Change from base Build 17910352266: 0.06%
Covered Lines: 5238
Relevant Lines: 6247

💛 - Coveralls

@himanshusinghs himanshusinghs merged commit 9f4c48b into main Sep 22, 2025
20 of 21 checks passed
@himanshusinghs himanshusinghs deleted the chore/update-readme branch September 22, 2025 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-title-validation Add this label to disable the title check for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants