Fix typo in Docker installation documentation - #48
Conversation
Krystropolis
left a comment
There was a problem hiding this comment.
The words are still misspelled here. "Official" is the correct spelling and it is misspelled twice in line 3.
gloriafolaron
left a comment
There was a problem hiding this comment.
Official is spelled incorrectly. Thank you for updating!
|
Maintainer review (advisory — merge authority stays with @marcelfolaron / @broskees) Intent: Fixes a typo in the Docker install intro line of Change requests (must address before merge):
Acceptance checklist (must pass before merge):
CI status: No test gate for docs. The blocker is correctness: as written this PR replaces one misspelling with another, so it should not merge until item 1 is fixed. |
|
Maintainer review — status: needs-info (advisory) · priority: P3 · next action: author completes the fix · owner: @RhesusP 1. Intent: Fixes a typo in 2. Change requests:
3. Acceptance checklist (must pass before merge):
4. CI status: Not readable through my current tooling — confirm green in the UI. No test gap (docs typo); the blocker above is content correctness. Advisory review only — not an approval. Merge authority stays with @marcelfolaron / @broskees. |
|
Maintainer review (advisory — merge authority stays with @marcelfolaron / @broskees) · first review · full diff read at head 1. Intent: Fixes a typo in 2. Change requests (the fix is incomplete — please correct before merge):
3. Acceptance checklist (must pass before merge):
4. CI status: No code/test surface — docs-only. No automated gate; the spelling correctness in CR #1–#2 is the merge gate (don't ship Advisory review only — not an approval, and I am not marking this ready or merging. Merge authority stays with @marcelfolaron / @broskees. |
|
Maintainer review (advisory — merge authority stays with @marcelfolaron / @broskees) · flagged in today's sweep as a long-open, never-reviewed PR (opened 2026-01-14, >14 days) from a first-time contributor · reviewed at head 1. Intent: Typo fix in the Docker installation documentation — "Offcicial" → "Official". 2. Change requests: None. Correct, trivial, user-visible spelling fix with no functional impact. Thanks, @RhesusP. 3. Acceptance checklist (must pass before merge):
4. CI status: No readable CI gate; prose-only. Blocker is only a maintainer merge — open since January 2026. Recommend batching with the other docs typo PRs (#44/#45/#46/#51) in one pass. Advisory only — not an approval. Merge authority stays with @marcelfolaron / @broskees. |
Offcicial -> Official