fix: drop hardcoded default for --min-group-size CLI flag - #312
Conversation
- Declare --min-group-size without default=100 so _resolve_opts can use profiler.MIN_GROUP_SIZE. - Matches how the other threshold flags already work for profile and compare. Fixes yakew7#299 Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
|
@VedantMadane is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Hey @VedantMadane, your first PR on Fair Code, that's awesome, thank you. This project is about making AI more accountable, and contributions like yours are what keep that work going. We'll review your changes shortly. If you haven't already, give the contributing guide a quick read: it covers how audits are structured and what we look for in a review. |
|
@yakew7 - new PR touching a path you own, please review. |
|
Hey @ahmdkaml could you just review this? meanwhile hey @VedantMadane! I'm yash the owner of this repo and i just wanna say thank you for the pr if u like this repo pls give it a star it helps other find this repo out asw and it helps me out also. If you're interested in the repo further prs are always appreciated |
yakew7
left a comment
yakew7
left a comment
There was a problem hiding this comment.
Edits seem fine and proper asw!
Credits AnayDhawan (#294, closing #283/#272), Shreyash0712's follow-up (#295), new contributor VedantMadane (#312, closing #299), propcgamer20-png (#311, closing #298), and ahmdkaml (#288, closing #270 - manually closed since the PR never referenced the issue). Reordered the Contributors section for AnayDhawan's new PR-count tier. Added the 2026-W35 metrics snapshot: stars 43->47, forks 22->23, contributors 17->18 (live GitHub numbers plus VedantMadane's first merged PR), synced across ROADMAP.md and README.md.
3 participants
Summary
fix: drop hardcoded default for --min-group-size CLI flag
Changes
Fixes #299