Skip to content

Conversation

@Lucoshi
Copy link

@Lucoshi Lucoshi commented Oct 7, 2025

Starting an update.

Summary by CodeRabbit

  • Documentation
    • Added an “Acronymous” column with populated acronyms for multiple worlds.
    • Introduced anchor links and emphasized acronyms within world entries for easier referencing.
    • Reorganized content from a flat list into per-world sections with clear headings and structured tables, improving readability and navigation.
    • Adjusted spacing and formatting across sections to ensure consistency.
    • No functional changes; content remains the same aside from presentation improvements.

@coderabbitai
Copy link

coderabbitai bot commented Oct 7, 2025

Walkthrough

The Worlds documentation file was reformatted: a new “Acronymous” column was added to the main table, entries gained anchors and emphasized acronyms, and the content was reorganized into per‑world sections with headings and local tables. Spacing and header adjustments were applied without altering underlying data beyond presentation.

Changes

Cohort / File(s) Summary of Changes
Docs: KH1 Worlds restructuring
docs/kh1/worlds.md
Added “Acronymous” column to main table; inserted anchors and emphasized acronyms in entries; reorganized content into per‑world sections with ## headings and per‑section Filename/Area tables; adjusted spacing/blank lines; presentation-only updates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I twitch my ears at tidy rows,
New headers bloom where table flows.
Acronyms shine—a crisp new gloss,
World by world, no data loss.
I hop through sections, neat and bright—
Docs aligned, a pleasing sight. 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title “Update worlds.md” correctly identifies the file being modified but is overly generic and does not convey the specific formatting enhancements such as adding an “Acronymous” column and restructuring content into per-world sections. Please revise the title to be more descriptive of the core changes, for example “Add Acronymous column and section headings to worlds.md” to clearly reflect the main updates.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b41b5b4 and 70f7a5b.

📒 Files selected for processing (1)
  • docs/kh1/worlds.md (11 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
docs/kh1/worlds.md

5-5: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


6-6: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


7-7: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


8-8: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


9-9: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


10-10: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


11-11: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


13-13: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


14-14: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


15-15: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


16-16: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


18-18: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


19-19: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


20-20: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


21-21: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


52-52: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


53-53: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


54-54: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


55-55: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


56-56: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


57-57: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


61-61: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


62-62: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


63-63: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


64-64: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


65-65: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


167-167: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


168-168: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


169-169: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


170-170: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


171-171: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


172-172: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


190-190: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


191-191: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


192-192: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


193-193: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


194-194: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)


195-195: Table pipe style
Expected: leading_only; Actual: leading_and_trailing; Unexpected trailing pipe

(MD055, table-pipe-style)

Comment on lines +3 to 21
| Id | World | Acronymous
|----|-------------------|---------
| AL | Agrabah
| AW | Wonderland
| DC | Disney Castle
| DH | Dive to Heart
| DI | Destiny Island
| EW | End of the World
| GB | Unknown
| HE | Olympus Coliseum
| HE | Olympus Coliseum ](#olympus)| **He**rcules
| LM | Atlantica
| NM | Halloweentown
| PC | Hollow Bastion
| PI | Monstro
| PO | 100 Acre Wood
| PO | 100 Acre Wood ](#100-acre-wood)| **Po**oh
| PP | Neverland
| TW | Traverse Town
| TZ | Deep Jungle
| ZZ | Miscellaneous
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Restore valid table formatting.

The added “Acronymous” column isn’t carried through the table: the header lacks the closing pipe, and every data row still has only two cells. Olympus Coliseum and 100 Acre Wood rows also have stray ](#...) fragments instead of proper column separators. As written, the Markdown table breaks (rendering collapses and markdownlint flags MD056/MD055). Please update the header and each row so all three columns are delimited by pipes and the anchor/acronym text sits entirely inside the third column.

🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

5-5: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


6-6: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


7-7: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


8-8: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


9-9: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


10-10: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


11-11: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


13-13: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


14-14: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


15-15: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


16-16: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


18-18: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


19-19: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


20-20: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)


21-21: Table column count
Expected: 3; Actual: 2; Too few cells, row will be missing data

(MD056, table-column-count)

🤖 Prompt for AI Agents
In docs/kh1/worlds.md around lines 3 to 21, the Markdown table is malformed: the
header is missing a trailing pipe and the separator row and each data row only
have two cells, plus two rows contain stray "](#...)" fragments; fix it by
restoring a three-column table (Id | World | Acronymous) with a proper separator
line containing three columns of dashes and pipes, then update every data row to
include three pipe-delimited cells (Id | World | Acronymous) moving the anchor
fragments and acronym text entirely into the third column (e.g., Olympus
Coliseum -> third column "He (link)" and 100 Acre Wood -> third column "Po
(link)"), ensuring every line starts and ends with a pipe and the columns are
consistent.

Copy link
Collaborator

@Delta-473 Delta-473 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix this by adding | to the ends of the rows because now you have a different amount of columns on the rows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants