Re: [PR] tests(issue-fix-workflow): eval suites for steps 3, 4, and 5 [airflow-steward]
potiuk commented on PR #239: URL: https://github.com/apache/airflow-steward/pull/239#issuecomment-4496457615 Damn . Too fast tried to see if I can speed up the merging before rename (and before @justinmclean goes to bed :) ... -- 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]
Re: [PR] tests(issue-fix-workflow): eval suites for steps 3, 4, and 5 [airflow-steward]
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]
Re: [PR] tests(issue-fix-workflow): eval suites for steps 3, 4, and 5 [airflow-steward]
potiuk merged PR #239: URL: https://github.com/apache/airflow-steward/pull/239 -- 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]
Re: [PR] tests(issue-fix-workflow): eval suites for steps 3, 4, and 5 [airflow-steward]
potiuk commented on PR #239: URL: https://github.com/apache/airflow-steward/pull/239#issuecomment-4496133972 More evals! -- 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]
