Repository navigation
Document Platform consumer release version ownership - #54
Conversation
|
👋 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
Reviewed the single AGENTS.md change. The documented ownership rule is clear, but it is not yet consistent with the current SDK synchronization workflow and SDK repository guidance, creating a concrete release-process regression risk.
💬 Posted 1 inline comment
⚠️ HIGHAGENTS.md:43This policy conflicts with the current SDK owner and automation.ultralytics/sdkCI still writespython.versionwhen the previous SDK version is ahead of the contract and independently increments the patch when generated output changes at an unchanged version; itsAGENTS.mdalso instructs maintainers to updatepython.versionfor intentional SDK changes. A maintainer following this new guidance can therefore still get an independently versioned SDK, or follow contradictory instructions a…
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 2
Made with ❤️ by Ultralytics Actions
Since the previous review, the SDK-side enforcement and guidance repair has merged, resolving the earlier workflow conflict and aligning automation with this policy. (github.com) The current AGENTS.md-only diff is consistent and introduces no genuine issues. LGTM.
|
🎉 PR #54 has been merged—thank you, @glenn-jocher!
This documentation makes that collaboration concrete by clarifying how the Platform API contract and |
Document the single release owner: the upstream Platform API contract's
info.versionmust equal the generated SDK package version. Explicitly prohibit SDK overrides, independent patch increments, and editing generated versions; give the release sequence for SDK-only fixes, wheel verification before consumer updates, and forward-only recovery from already-published mismatches.The enforcement and SDK guidance change is ultralytics/sdk#75; merge this documentation after that repair so instructions and automation agree.
Documentation only; reviewed against the API contract and SDK synchronization/publishing workflows, with
git diff --checkpassing.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Documented that the upstream Platform API contract owns the
ultralytics-platformSDK version, ensuring both versions remain synchronized through the documented release workflow.📊 Key Changes
AGENTS.mdrequiring the SDK version to match the API contract’sinfo.version.python.version, independent SDK patch bumps, and hand-editing generated versions.🎯 Purpose & Impact