Skip to content

PR status dashboard using Github pages#4779

Open
CharlieL7 wants to merge 11 commits intodevelopfrom
pr_dashboard
Open

PR status dashboard using Github pages#4779
CharlieL7 wants to merge 11 commits intodevelopfrom
pr_dashboard

Conversation

@CharlieL7
Copy link
Copy Markdown
Collaborator

@CharlieL7 CharlieL7 commented Apr 10, 2026

Motivation

  • Make it easier to figure out what the status of PRs are for the MIGraphX project

Technical Details

  • When live this page should be available at the Github Pages link
  • Should run automatically every hour or if the github action is manually triggered.
  • Currently uses my PAT I've set as a repo secret that will expire in 30 days. Should probably be using someone else's PAT.
  • Groups open PRs by if they have been reviewed by 2+ AMDMIGraphX members with write access to this repo.
  • Add a dashboard note by adding a comment to the PR with the #dash_note prefix.
  • Made categories at the top of the page clickable to link to the table.
image

Changelog Category

Add a CHANGELOG.md entry for any option other than Not Applicable

    • Added: New functionality.
    • Changed: Changes to existing functionality.
    • Removed: Functionality or support that has been removed. (Compared to a previous release)
    • Optimized: Component performance that has been optimized or improved.
    • Resolved Issues: Known issues from a previous version that have been resolved.
    • Not Applicable: This PR is not to be included in the changelog.

@CharlieL7 CharlieL7 self-assigned this Apr 10, 2026
@CharlieL7 CharlieL7 added the skip bot checks Skips the Performance and Accuracy CI tests label Apr 10, 2026
@CharlieL7 CharlieL7 marked this pull request as ready for review April 10, 2026 19:53
@CharlieL7 CharlieL7 requested a review from causten as a code owner April 10, 2026 19:53
Copilot AI review requested due to automatic review settings April 10, 2026 19:53
@pfultz2
Copy link
Copy Markdown
Collaborator

pfultz2 commented Apr 10, 2026

Groups open PRs by if they have been reviewed by 2+ AMDMIGraphX members with write access to this repo.

Does this exclude the documentation team reviewers?

Copy link
Copy Markdown
Contributor

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

Adds an automated PR review-status dashboard for ROCm/AMDMIGraphX intended to be published via GitHub Pages, grouping open non-draft PRs by reviewer/approval/CI status and supporting “#dash_note” annotations.

Changes:

  • Added a Python data generator that queries GitHub APIs and emits a categorized JSON report.
  • Added a static HTML dashboard that renders data.json into summary cards and bucket tables.
  • Added a GitHub Actions workflow intended to generate data.json hourly and deploy the dashboard to GitHub Pages.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
tools/pr-review-status.py Collects PR/review/CI/comment data and outputs a structured JSON report for the dashboard.
pr_review_dashboard/index.html Renders the JSON report into a GitHub Pages-friendly dashboard UI.
.github/pr-review-status.yaml Intended scheduled/manual Pages deploy workflow to refresh and publish the dashboard.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +30 to +36
- run: pip install requests

- name: Generate data.json
env:
GITHUB_TOKEN: ${{ secrets.PAT_TOKEN }}
run: python tools/pr-review-status.py --json > pr_review_dashboard/data.json

Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

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

This workflow relies on a long-lived PAT stored in secrets.PAT_TOKEN to query PR/review data. For least-privilege and easier rotation, prefer using the built-in GITHUB_TOKEN (and add the minimal required permissions: such as pull-requests: read) or a GitHub App / fine-grained token scoped to read-only repo/PR metadata.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It needs PAT because the GITHUB_TOKEN doesn't have enough access. We can do a fine-grained token, but it would have to be from the ROCm user who owns the AMDMIGraphX repo

@CharlieL7
Copy link
Copy Markdown
Collaborator Author

Groups open PRs by if they have been reviewed by 2+ AMDMIGraphX members with write access to this repo.

Does this exclude the documentation team reviewers?

It should exclude them if they do not have push/write access to the repo. If they do have access we would have to maintain a list of valid reviewers.

This comment was marked as outdated.

@CharlieL7 CharlieL7 requested a review from pfultz2 April 10, 2026 21:15

This comment was marked as outdated.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4779      +/-   ##
===========================================
+ Coverage    92.45%   92.49%   +0.04%     
===========================================
  Files          583      583              
  Lines        29298    29515     +217     
===========================================
+ Hits         27085    27297     +212     
- Misses        2213     2218       +5     

see 13 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip bot checks Skips the Performance and Accuracy CI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants