V2.2.0 - #1
Conversation
Every run warned that checkout, configure-pages and upload-artifact target a runtime GitHub now forces to Node 24, and will eventually stop supporting. upload-pages-artifact needs v5 rather than v4: it is a composite action, and v4 still calls upload-artifact 4.6.2, which is itself Node 20. Bumping only to v4 would leave the warning in place. Checkout goes to v5 rather than the current v7. All three of v5, v6 and v7 run on Node 24, so they are equally durable here, and the step passes no options. Side effect: v4 onwards excludes dotfiles from the artifact, so .gitignore stops being served. It should never have been.
Bumps every stated version: the hero badge and its link, the "Latest release" fact, the structured data, both console mocks, and the Helm card, which moves to chart 0.6.3 for app version 2.2.0. Adds a release card alongside 2.0.0 and 2.1.0, so the row now holds three. It leads with the Milvus 3.0 upgrade and says plainly that existing 2.x deployments must be migrated first, and that BM25 turning case-insensitive needs a schema migration too. Someone reading a release note should learn that before they upgrade, not after. Milvus is named on the page but never versioned, so nothing else needed correcting for the 3.0 move.
ReviewChecked the version bumps against the upstream release and the action bumps against the published action versions. Verified correct
1. Stale meta description —
|
Every other version surface moved to 2.2 with the v2.2.0 update, but the plain meta description still advertised 2.1. That string is what search engines render in the result snippet, so the listing contradicted og:description, the JSON-LD softwareVersion and the hero.
Both breaking changes in the release card told operators a migration was required without giving them any way to reach it. The asset URL stays pinned to /download/v2.2.0/ rather than /releases/latest/download/: the card is a permanent record of that release, and latest would silently repoint it at a later version's assets.
The System status mock used pill-red for "Up", which is the brand and danger fill, so a healthy API read as an alert. The equivalent mock in the Overview panel already uses pill-ok for the same value.
|
Thanks — all three actionable points are now fixed on the branch (d1af152, dbb1ab1, 7fe8a80). 1. Stale meta description — fixed. 3. Migration links — done, using your snippets. Both bullets now link out: the Milvus 3.0 bullet to the pinned release asset 4. Nit — fixed. 2. sitemap 5. Callback URLs — leaving the card at five bullets. It is already the longest of the three, and the two breaking changes plus the three headline features are what an operator needs from a release card. The full changelog link below the cards covers the rest. |
The sitemap still advertised 31 July while the page itself changed with the v2.2.0 update. Crawlers use lastmod to decide when to revisit, so a stale value delays the refreshed listing.
|
Follow-up on item 2: the sitemap is done after all, in 0be8c30. That leaves only item 5 (callback URLs) deliberately not taken. |
andyne13
left a comment
There was a problem hiding this comment.
All five review items are resolved. Verified at 0be8c30:
- Meta description now advertises 2.2, and no stale
2.1/2.1.1reference remains anywhere in the page or the sitemap. sitemap.xmllastmodrefreshed to2026-09-04, and the file is well-formed.- Migration links — the Milvus 3.0 and BM25 bullets now reach the package and the docs. The release asset and the documentation page both return 200, the
#schema-migrationsanchor genuinely exists on that page, and the asset URL is correctly pinned to/download/v2.2.0/rather thanlatest, so the card keeps pointing at the 2.2.0 package after 2.3 ships. - Console mock — both System status blocks now use
pill-okfor "Up". - The callback-URLs bullet was left out, which is a fair editorial call on card length.
index.html still parses with no tag-nesting errors. The CI bumps are unchanged from the earlier check: checkout@v5, configure-pages@v6 and upload-pages-artifact@v5 are all node24 and compatible with the deploy-pages@v5 already on main.
Content matches the upstream v2.2.0 release, and the chart line matches infra/charts/openrag-stack/Chart.yaml at that tag. LGTM.
No description provided.