build(evaluators): standardize contrib packaging and release flow#193
build(evaluators): standardize contrib packaging and release flow#193
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Template scaffolds stale The rest of the PR moves in-repo contrib packages onto (Posting as a general PR comment rather than inline because those two dependency lines are unchanged context between diff hunks in this PR, so GitHub will not accept an inline review comment there.) Consider either (a) a literal placeholder like |
|
Addressing #193 (comment): I took the placeholder route here. The contrib template now uses |
|
Since I replied on the review threads, I cleaned up a few more things:
I also ran a fresh review loop after that. No new blocking findings came back. The remaining nits are cleanup-level, not merge blockers. |
Summary
agent-control-evaluators[...]and bring package-local checks into line across the in-repo contrib evaluatorsuv.sources, and extras wiring stay in syncReview Order
build(evaluators): normalize contrib package wiringfix(evaluators): satisfy budget package-local checksbuild(evaluators): normalize package-local Makefile targetsbuild(infra): add contrib package discovery scriptci(infra): drive contrib releases from discoveryci(infra): make root checks contrib-awarechore(evaluators): refresh contrib template packaging scaffoldfix(infra): address contrib packaging review feedbackfix(ci): harden contrib release and coverage flowfix(ci): stabilize release artifact publishingtest(evaluators): cover contrib version fallbacksDesign Context
Validation
make syncmake contrib-verifymake scripts-testmake test-extrasmake contrib-lintmake contrib-typecheckmake contrib-builduv run python scripts/build.py allmake check(stops inserver-testlocally because Postgres is not listening onlocalhost:5432)pg_isready -h localhost -p 5432 -U agent_control->no response