andreahlert commented on PR #239: URL: https://github.com/apache/airflow-steward/pull/239#issuecomment-4496238940
@potiuk I was still going through this when it got merged 😄, but flagging now because the squash dragged in scope that wasn't in the PR description. the branch was cut off `contribitor-readiness` (#227) instead of `main`, so the squash merge landed the entire `contributor-nomination` skill into main as part of this PR. 87 files in `c11f13c`, of which only ~25 are the step-3/4/5 evals this PR actually describes. the other ~60 are the SKILL.md + assess/fetch/render + the full `contributor-nomination` eval suite + `docs/modes.md` edit + the project template. confirmed by diffing the merge against its parent `76dcb977`: `.claude/skills/contributor-nomination/` didn't exist there, it does now. side effect: #227 is still open but its content is already on main, never got its own review. happy to draft a follow-up that closes #227 and patches the spec nits I caught in the step-3/4/5 fixtures (verdict rubric ambiguity on step-3, missing edge case on step-5, schema overlap between step-4 `in_scope` and step-6) if useful. -- 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]
