-
Couldn't load subscription status.
- Fork 86
feat: Add PR comment statistics behind HIDE_PR_STATISTICS configuration flag #604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add PR comment statistics behind HIDE_PR_STATISTICS configuration flag #604
Conversation
…guration Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
…e effects Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a PR comment statistics feature that can be toggled via the HIDE_PR_STATISTICS configuration flag. The feature displays mean, median, and 90th percentile comment counts for pull requests, helping identify PRs that may need attention due to high comment activity.
Key Changes:
- Adds a new
pr_comments.pymodule for counting PR comments (excluding bot comments and ignored users) - Extends configuration system with
HIDE_PR_STATISTICSenvironment variable (defaults to true) - Integrates PR comment statistics into both Markdown and JSON output formats
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
pr_comments.py |
New module implementing PR comment counting and statistics functions |
config.py |
Adds HIDE_PR_STATISTICS configuration option to environment variables |
classes.py |
Extends IssueWithMetrics class with pr_comment_count field |
issue_metrics.py |
Integrates PR comment counting into metrics collection pipeline |
markdown_writer.py |
Updates Markdown output to include PR comment statistics and individual counts |
json_writer.py |
Updates JSON output to include PR comment statistics |
README.md |
Documents the new HIDE_PR_STATISTICS configuration option |
test_*.py |
Comprehensive test coverage for new functionality and updates to existing tests |
|
@copilot can you fix the failing actions on this pr by running prettier on the markdown files and making the proper fixes? |
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.