potiuk opened a new pull request, #106:
URL: https://github.com/apache/airflow-steward/pull/106

   ## Summary
   
   Two doc-quality fixes for the `pr-management-stats` skill, mirroring review 
feedback from @choo121600 on the equivalent `apache/airflow` PR 
([apache/airflow#66464](https://github.com/apache/airflow/pull/66464)).
   
   ### 1. Anchor links drop the fragment in the URL
   
   Cross-references like ``[`render.md#recommendation-rules`](render.md)`` show 
an anchor in the visible text but the URL omits it, so GFM never scrolls to the 
target section. 26 of these across 5 files. Fix is mechanical: copy the visible 
anchor into the URL.
   
   Three target headings were tweaked so the GFM-generated anchors actually 
exist (the original link texts were short forms that didn't match the heading 
slugs):
   
   - `aggregate.md`: link `aggregate.md#counters` → 
`aggregate.md#counters-per-area` (matches heading "Counters (per area)").
   - `fetch.md`: rename `Known limitation — GitHub search-index lag for 
closed-since counts` → `Known limitation`.
   - `fetch.md`: rename `Closed / merged triaged PRs (since cutoff)` → 
`Closed-merged-triaged PRs`.
   
   ### 2. `Action` column conflated paste-clean commands with prose
   
   The recommendation-rules table in `render.md` had entries like 
`/pr-management-triage stale → sweep 1a`, `No immediate action — check next 
week.`, and `review /pr-management-triage stale cadence — too many PRs reaching 
the sweep` in the `Action template` column — directly contradicting the rule on 
the same page that says *"the action template must be a literal slash-command 
the maintainer can paste"*.
   
   Split into two columns:
   
   - `Action` — literal paste-clean slash command, or `—` when no command 
applies (rules 8 and 10).
   - `Detail` — prose explanation that goes in the card body.
   
   The card-rendering description in `render.md` and the schema description in 
`SKILL.md` step 5a are updated to match (cards omit the code block when 
`action` is `—`).
   
   related: [apache/airflow#66464](https://github.com/apache/airflow/pull/66464)
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.7)
   
   Generated-by: Claude Code (Opus 4.7)


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to