You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/06/04 01:39:02 UTC
[PR] pr-management-stats: per-chart legends, non-draft backlog, maintainer-opened panel, distinct area colours, real pressure ages [airflow-steward]
potiuk opened a new pull request, #447:
URL: https://github.com/apache/airflow-steward/pull/447
Improvements to `tools/pr-management-stats/dashboard.py`, found while running the dashboard against `apache/airflow`:
1. **Per-chart legends, no bottom legend.** Drop the single `Legend / methodology` panel (and the now-unused `render_legend`); every chart already carries its own inline legend / caption.
2. **Open-backlog: add a `non-draft` line** alongside the total backlog line.
3. **New "Maintainer-opened PRs" panel** (below *PRs opened by author class*) — splits maintainer-authored PRs (authorAssociation OWNER/MEMBER/COLLABORATOR) by **author** and by **`provider:*` area**.
4. **Distinct per-area colours** for the top-areas ready-for-review trend chart (new `AREA_PALETTE`). The previous pressure-band colours (red/amber/grey) repeated across areas and were indistinguishable.
5. **Fix the "pressure by area" age columns** (`>4w` / `1-4w` / `recent`). They only counted `_is_untriaged` PRs, so with almost everything triaged they showed ~0 everywhere. They now show the **age distribution of each area's open non-draft contributor PRs**. The pressure *score* and recommendations still use the untriaged-age weighting (unchanged).
6. **Rename** the ready-queue series label `ready cum` → `ready queue`.
## Test plan
- `python3 -m pytest tools/pr-management-stats/tests/` — all 24 pass.
- Rendered against `apache/airflow` (485 open PRs); verified each fix in the output: non-draft backlog line present, maintainer panel populated (601 maintainer PRs split by author + provider), pressure age columns now non-zero (e.g. providers 26 >4w / 36 1-4w / 22 recent), distinct area hues, `ready queue` label, no bottom legend.
- No new `ruff` findings (this file is not a `[tool.ruff]` workspace member, so it is not reformatted; the diff is logical-only).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] pr-management-stats: per-chart legends, non-draft backlog, maintainer-opened panel, distinct area colours, real pressure ages [airflow-steward]
Posted by "github-advanced-security[bot] (via GitHub)" <gi...@apache.org>.
github-advanced-security[bot] commented on code in PR #447:
URL: https://github.com/apache/airflow-steward/pull/447#discussion_r3352964920
##########
tools/pr-management-stats/dashboard.py:
##########
@@ -1346,17 +1460,20 @@
)
out = [
"<h2>Pressure by area</h2>",
- '<div class="caveat">Pressure score = weighted sum of urgent PR conditions per area. Higher score = more attention needed.</div>',
+ '<div class="caveat">Pressure score = weighted sum of urgent PR conditions per area '
+ "(untriaged >4w ×5, 1–4w ×3, <1w ×1; triaged-waiting >7d ×2; ready ×1). "
+ "Age columns show the age distribution of the area's open non-draft contributor PRs "
+ "as <strong>ready/all</strong> (PRs labelled ready-for-review / total in that age bucket).</div>",
Review Comment:
## CodeQL / Implicit string concatenation in a list
Implicit string concatenation. Maybe missing a comma?
[Show more details](https://github.com/apache/airflow-steward/security/code-scanning/15)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] pr-management-stats: per-chart legends, non-draft backlog, maintainer-opened panel, distinct area colours, real pressure ages [airflow-steward]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #447:
URL: https://github.com/apache/airflow-steward/pull/447
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] pr-management-stats: per-chart legends, non-draft backlog, maintainer-opened panel, distinct area colours, real pressure ages [airflow-steward]
Posted by "github-advanced-security[bot] (via GitHub)" <gi...@apache.org>.
github-advanced-security[bot] commented on code in PR #447:
URL: https://github.com/apache/airflow-steward/pull/447#discussion_r3352919523
##########
tools/pr-management-stats/dashboard.py:
##########
@@ -1346,17 +1450,19 @@
)
out = [
"<h2>Pressure by area</h2>",
- '<div class="caveat">Pressure score = weighted sum of urgent PR conditions per area. Higher score = more attention needed.</div>',
+ '<div class="caveat">Pressure score = weighted sum of urgent PR conditions per area '
+ "(untriaged >4w ×5, 1–4w ×3, <1w ×1; triaged-waiting >7d ×2; ready ×1). "
+ "Age columns show the age distribution of the area's open non-draft contributor PRs.</div>",
Review Comment:
## CodeQL / Implicit string concatenation in a list
Implicit string concatenation. Maybe missing a comma?
[Show more details](https://github.com/apache/airflow-steward/security/code-scanning/14)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org