Skip to content

Feat/configurable GitHub query limits#165

Closed
ShashwatUpadhayay wants to merge 2 commits into
O2sa:mainfrom
ShashwatUpadhayay:feat/configurable-github-query-limits
Closed

Feat/configurable GitHub query limits#165
ShashwatUpadhayay wants to merge 2 commits into
O2sa:mainfrom
ShashwatUpadhayay:feat/configurable-github-query-limits

Conversation

@ShashwatUpadhayay

Copy link
Copy Markdown
Contributor

Summary

  • Make GitHub query limits configurable through environment variables:
    • GITHUB_REPO_COUNT
    • GITHUB_PR_COUNT
    • GITHUB_ISSUE_COUNT
    • GITHUB_DISCUSSION_COUNT
  • Add parsing and validation with sensible defaults and upper bounds
  • Replace hardcoded query limits with runtime configuration
  • Document the new environment variables
  • Add unit tests for parsing and validation logic

Testing

  • Verified default values are used when environment variables are missing or invalid
  • Verified valid values are parsed correctly
  • Verified oversized values are clamped to the configured maximum
  • Ran the test suite successfully

@vercel

vercel Bot commented Jun 21, 2026

Copy link
Copy Markdown

@ShashwatUpadhayay is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

@ShashwatUpadhayay

Copy link
Copy Markdown
Contributor Author

Hello.. Please merge this PR if it is all good... Please notify in case of any errors

@O2sa

O2sa commented Jun 22, 2026

Copy link
Copy Markdown
Owner

You should do your new edits in the top of the latest changes of the main branch and to do that go to the main branch and pull the latest changes and then create your new branch from the main...

@ShashwatUpadhayay

Copy link
Copy Markdown
Contributor Author

My Apologies!.. Issuing a new PR with the above fixes
Thanks!.

@ShashwatUpadhayay

Copy link
Copy Markdown
Contributor Author

Close #160

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.

2 participants