Repository navigation
docs: external Redis must be 6.2 or later; /main/ canonical to /latest/ - #239
Merged
Merged
Conversation
main stays noindex. scripts/check-canonical.mjs, run on every PR for both main and latest, fails on a page without exactly one canonical to its /latest/ counterpart.
Contributor
Author
|
APPROVED at 91d047a, both commits. The canonical from every /main/ page to the same page under /latest/ matches the live /latest/ URL shape (index.md to the directory, the rest .html), noindex stays on main, and the check runs on both builds in CI and is green. The script's failing cases were exercised. No version bump: docs only. The PR description still speaks only of the Redis minimum; add one line about the canonical change before merging. |
0kaba0hub
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Redis minimum version
The yarilo Redis dict now reads a path through a key index with
ZRANGE … BYLEX(yarilo#2390), which needs Redis 6.2. The public docs named no minimum Redis version at all. This adds a Redis subsection to the Helm prerequisites.Canonical for /main/
Pages under
/main/hadnoindexbut no canonical. Every/main/<path>page now also carries<link rel="canonical" href="https://doc.yarilomail.org/latest/<path>">, the same page under/latest/.noindexstays. Releases and/latest/are unchanged (/latest/is canonical to itself).scripts/check-canonical.mjsfails on any built page without exactly one canonical to its/latest/counterpart. The PR check runs it onsite/mainand onsite/latest.Docs-only, no version bump.
npm run docs:buildpasses.