docs: mirror freeunitorg/docs#9 for migration-guide review#1
Draft
andypost wants to merge 3 commits into
Draft
Conversation
added 3 commits
April 18, 2026 23:05
- New migration.rst with Docker, source build, and community repo instructions - Compatible with FreeUnit v1.35.3+ - Follows installation.rst style guide
migration.rst: split guide into user vs. maintainer audiences; rewrite Community Repository Maintainers section with per-distro packaging recipes (Alpine, ALT, Arch, FreeBSD, Gentoo, NetBSD, Nix, OpenBSD, Remi RPM); fix Docker section — correct tag format (<VERSION>-<VARIANT>), add full variant table from docker.yml (27 variants), add language EOL policy note, fix rollback example tag, fix verify command. installation.rst: add "coming soon" warnings for official packages, link to migration guide, update Docker section to GHCR registry with correct tag format and variant list, mark repo install script and Homebrew tap as archived/incompatible, fix all residual nginx/unit source URLs.
Owner
Author
|
Despite the successful resolution of the major problems, a few items still need attention: The next step should be to fix the truncated file issue, make a final pass on the remaining nits, and then re-request a build to ensure Sphinx runs with zero warnings. Let me know if you'd like me to elaborate on any of these points. |
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.
Summary
This is a local mirror of freeunitorg#9 so the review feedback there can be addressed as small, reviewable commits before being sent upstream. GitHub MCP access in my environment is scoped to
andypost/unit-docs, so the fix iteration happens here.What was done
upstream = https://github.com/freeunitorg/docs.gitand fetched the PR branch (also available asrefs/pull/9/head).masterfrom nginx6c3106c→ freeunitorg33fd8b9(479 commits, all fromfreeunitorg/main). This is the PR's actual base — without it the migration guide has 47 "undefined label" warnings against anchors that freeunitorg defines in new files (community.rst,certificates.rst, etc.) and 1 invalidmetadirective ERROR.claude/review-and-fix-pr-chDsnto the PR tipd5e797d. This branch is now exactly the 3 PR commits on top of master:c513e83— docs: add migration guide25575c6— docs: rewrite migration and installation for FreeUnit forkd5e797d— docker: Build local for testBoth FFs are non-destructive (verified with
git merge-base --is-ancestor).Baseline build numbers (Sphinx 6.2,
sphinx-build -E -n)migration.rst33fd8b9(master/freeunit main)c513e83(add migration guide)25575c6(rewrite)d5e797d(tip)The 9 baseline warnings are pre-existing
envvartargets onfreeunitorg/main— out of scope. The 47 new warnings are the ones the upstream review calls out.Next commits on this branch
Following the review of freeunitorg#9, subsequent commits on this branch will each address one concern so diffs stay small:
.. meta::indentation → clears the ERROR.\*BSD→ clears the "inline emphasis" warning.socket,file,user and group,TLS,regex,community,migration-remi-hybrid) → clears ~40 warnings.migrationintocontents.rsttoctree → orphaned-doc warning.unitd --versionoutput,./configure --modules=invalid syntax,unitd --test-confignon-existent,make installunitctl claim,pkg/systemd/unit.servicemissing file, Docker matrix variants that don't exist.docker-composev1 withdocker composev2.migration-remi-hybridsection (defines the dangling anchor).Verification loop (autonomous)
-W --keep-goinggives a non-zero exit on any residual warning and a complete list in one pass, so each fix commit is either clean or obviously regressing.This PR is opened as draft because no fixes have landed yet — it's the starting point for the fix sequence described above.