Truncate issue#32
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds configurability to prevent the Pull Request Dashboard issue body from exceeding GitHub’s issue-body size limit in very large repositories by allowing row truncation per section and optionally omitting drafts.
Changes:
- Introduces
max_rows_per_sectionto cap rendered rows per routing section (and drafts) and adds a “More X PRs not shown” truncation note. - Introduces
skip_draftsto optionally omit the Draft pull requests section entirely. - Wires both options through the reusable workflow inputs, repository configuration, CLI flags, and rendering logic.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pull-request-dashboard/README.md | Documents new max_rows_per_section and skip_drafts configuration options. |
| .github/workflows/pull-request-dashboard.yml | Passes new matrix options through to the reusable workflow. |
| .github/workflows/pull-request-dashboard-repo.yml | Adds workflow_call inputs and forwards them into the dashboard CLI invocation. |
| .github/scripts/pull-request-dashboard/repositories.json | Enables truncation + draft skipping for opentelemetry-collector-contrib. |
| .github/scripts/pull-request-dashboard/render.py | Implements row limiting + truncation note; adds optional draft skipping. |
| .github/scripts/pull-request-dashboard/dashboard.py | Adds CLI flags and forwards them into rendering. |
Comments suppressed due to low confidence (1)
.github/scripts/pull-request-dashboard/dashboard.py:1140
--max-rows-per-sectionaccepts negative values today; because the value is later used for slicing, a negative number would drop rows from the end and compute an incorrect truncation count. Since the flag documents0 = no limit, reject values < 0 during argument validation.
args = parser.parse_args()
if args.required_approvals < 1:
parser.error("--required-approvals must be at least 1")
with state_branch.temporary_state_dir() as state_dir:
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mx-psi
approved these changes
Jul 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Workflow is failing for large repo with hundreds of PRs (e.g.
opentelemetry-collector-contrib), because of the limit of characters on the issue body.This PR adds an option to limit the amount of PRs per section and also to skip draft entirely.