Bump specification stable to v0.9 and draft to v0.10#1419
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the A2UI documentation to promote version 0.9 to stable and introduce version 0.10 as a draft. The changes include new specification files, updated protocol examples, and a reorganization of the documentation navigation structure. Review feedback identified several technical inaccuracies in the documentation, specifically regarding incorrect version references in section headers, copy-paste errors in legacy protocol descriptions, and malformed or missing examples for version 0.8.
selamw1
force-pushed
the
stable_ver_bump
branch
3 times, most recently
from
86f0f0f to
ac352f8
Compare
ditman
left a comment
ditman
left a comment
There was a problem hiding this comment.
I haven't reviewed the whole thing, but I think this is breaking the documentation by changing the header versions without moving the json messages around.
A simpler fix may be to just update the adjective next to the version numbers, so something like:
=== "v0.8 (Stable)"Would get changed to:
=== "v0.8 (Legacy)Then if we want to move the code blocks around, it should be easy.
Who'd be best to review this further, @zeroasterisk?
Agreed. I have aligned the version tabs with their correct payload versions by updating only the version adjectives (e.g., changing Additionally:
|
ditman
left a comment
ditman
left a comment
There was a problem hiding this comment.
This LGTM! Thanks for the changes!
|
Hi all, I didn't see this PR and opened #1603 today. I diffed the two and there is definitely overlap around the broad v0.8/v0.9 status cleanup in the README and docs wrappers. The specific gaps I see are that this PR does not cover the remaining v0.9.1 current-production corrections. In particular, #1603 updates Suggest we either fold those remaining v0.9.1-specific fixes into this PR, or this PR can land first and I can narrow/rescope #1603 as a follow-up that only addresses the remaining v0.9.1 current-production references and examples. Either is fine with me! |
|
OK @samjulien, I landed this because it had been brewing for a while already. Feel free to rebase your PR and add the missing details there! Thanks for your contribution (and patience), @selamw1! |
3 participants
Bumps the A2UI documentation and specification status to reflect version 0.9 as the latest stable release, version 0.10 as the active draft, and version 0.8 as legacy.
Proposed Changes
docs/specification/):mkdocs.yaml): Re-ordered navigation hierarchy under the Specifications node to place v0.9 (Stable) at the top.README.md): Updated version warning notice in root landing page to v0.9.Ref/Closes: #1374
Pre-launch Checklist