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

   ## Summary
   
   Add a Before-submitting checklist entry that requires running \`lychee 
--config .lychee.toml .\` locally on every PR. Caught a Fragment-not-found 
regression in [PR #410](https://github.com/apache/airflow-steward/pull/410) 
only after CI flagged it; the check is cheap to run locally and catching it 
before the push avoids a round-trip.
   
   ## Change
   
   Append to the existing **Before submitting** checklist in \`AGENTS.md\`:
   
   1. The canonical invocation (matches the CI workflow).
   2. A note on the most common failure class — \`Fragment not found in 
document\` — which is exactly what bites on doc refactors that move sections 
between files or rename headings (the class that just bit PR #410).
   3. A one-liner workaround for v0.24+ local lychee installs, since 
\`.lychee.toml\` pins the v0.23 schema for CI compatibility.
   
   The rule is enforced by CI either way (the lychee workflow blocks merge on 
any failure); this just makes the local check a documented pre-push habit so 
agents avoid the CI round-trip.
   
   ## Test plan
   
   - [x] AGENTS.md renders cleanly (table of contents intact, no broken links 
introduced by this edit)
   - [ ] Next PR opened against the framework after this lands has the lychee 
local-run done before push (verified by absence of doc-refactor-induced 
fragment errors on CI)
   
   🤖 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: [email protected]

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

Reply via email to