Repository navigation
Refresh AGENTS.md against current code and CI - #57
Conversation
Remove the claim that Ruff formats Google-style docstrings (the generator emits them; format.yml sets python_docstrings: false), drop duplicated determinism and pull --rebase lines, condense the Platform SDK version policy that ultralytics/sdk AGENTS.md owns in full, list CI's pinned Ruff and compileall checks, and point to lib/config.ts and scripts/headers.ts.
|
👋 Hello @glenn-jocher, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
LGTM. Reviewed the AGENTS.md updates against the README, CI workflow, package scripts, and referenced repository locations. The guidance aligns with the current repository; no actionable issues found.
|
🎉 This PR has been merged—thank you, @glenn-jocher, for updating
|
Refresh
AGENTS.mdagainst current code and CI (docs only, net -8 lines). No code bugs found during the review.lib/generators/python.ts) andformat.ymlsetspython_docstrings: false. Removed; the Python output conventions section already covers docstrings.nodefor package scripts. Changed to Bun, uv, and Python.python -m compileall -q generated/python/srcand the pinneduvx ruff@0.16.2 check generated/python, but the command block left them out. Added both, and removed the prose that duplicated them.lib/config.ts(config schema, required keys,OPENAPI_CONFIG, relative-path resolution) andscripts/headers.ts.format.ymlPrettier/git pull --rebaseline (PR Workflow steps 1 and 4 cover it). Condensed the Platform SDK version section to its binding rules plus a pointer toultralytics/sdk'sAGENTS.md, which already holds the full procedure word for word.Validated:
bun run generate,bun run lint,bun run test(20 pass),compileall,ruff@0.16.2 check, andprettier@3.8.5 --print-width 120 --check AGENTS.md.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Updated
AGENTS.mdto match the repository’s current configuration, tooling requirements, and CI checks.📊 Key Changes
🎯 Purpose & Impact