Skip to content

Update Refactor Requirements: adds code note guidances for multiple game files and moves doc from content to dev section - #440

Draft
suspect15 wants to merge 2 commits into
RetroAchievements:mainfrom
suspect15:qarefactorupdate
Draft

suspect15 wants to merge 2 commits into
RetroAchievements:mainfrom
suspect15:qarefactorupdate

Conversation

@suspect15

Copy link
Copy Markdown
Contributor

Adds guidance for how to note sets with multiple game files as well as instances were the same address is used for different purposes on different game files.

@suspect15
suspect15 marked this pull request as draft October 14, 2026 01:28
@greptile-apps

greptile-apps Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Low impact] Moves and expands developer documentation about code notes.

Fix the broken sidebar link before merging.

Summary

Adds code-note guidance for multiple game files and regions.

  • Code notes mark which game file or region they describe.

Reviews (1) · Last reviewed commit: "Update: adds guidances for multiple game..." · Reviewed by Greptile

@greptile-apps

greptile-apps Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Comments Outside Diff

These findings could not be posted inline.

  • P1 Sidebar opens a missing page docs/guidelines/developers/refactor-requirements.md:1 ▶

    Moving this page leaves the sidebar pointing to /guidelines/content/refactor-requirements, which no longer exists. Readers clicking “Refactoring Requirements” reach a missing page instead of this guide.

    Suggestion: Move the sidebar entry to Developer Guidelines and update its URL.

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.

1 participant