Skip to content

Fix typos in comments, README, and addon manifests - #135

Merged
gantoine merged 1 commit into
rommapp:mainfrom
Avicennasis:docs/fix-typos
Sep 8, 2026
Merged

gantoine merged 1 commit into
rommapp:mainfrom
Avicennasis:docs/fix-typos

Conversation

@Avicennasis

Copy link
Copy Markdown
Contributor

Summary

Seven single-word spelling/wording fixes. No identifiers, no API surface, no behaviour changes.

Comments

  • Downloads/SidebarItemControl.cs: // Link contener -> // Link container
  • RomM.cs: // If Siblings are avaiable prompt user -> available
  • Games/RomMImport.cs: // Fail-safe incase none of these are set to true -> in case

User-facing text

  • README.md: create Playnite library entires -> entries
  • README.md: requires a RomM instance setup with IGDB API credentials -> set up with (verb, not noun)
  • README.md: how to set generate and set credentials -> how to generate and set credentials (stray duplicated set)
  • manifests/addon.yaml: same entires -> entries in the add-on description shown in Playnite's browser
  • manifests/installer.yaml: Fetch games in pagniated chunks of 72 -> paginated (0.4.0 changelog entry). Drop this hunk if you would rather leave a shipped changelog entry untouched.

Deliberately NOT changed

  • PART_GridContener in Downloads/SidebarItemControl.cs is misspelled, but it is a private property referenced in four places - renaming it is a code change, not a typo fix, so it is left for you to decide.
  • SiderbarItemType in Downloads/RomMDownloadsSidebarItem .cs is spelled that way because that is the actual enum name in the Playnite SDK (PlayniteSDK/Plugins/SidebarItem.cs). "Correcting" it would not compile.
  • The // ... revert any changes made to Option1 and Option2 comments in Settings/Settings.cs are Playnite's ISettings template boilerplate and reference example names that do not exist here; rewriting them is a judgement call for a maintainer, not a typo fix.
  • British spellings throughout (normalise, initialises, centralises, honour, favourites, centre, modelled) are consistent repo-wide and were left alone.

Spelling/wording only - no identifier, API or behaviour changes.

Signed-off-by: Avicennasis <avicennasis@gmail.com>
@gantoine

gantoine commented Sep 8, 2026

Copy link
Copy Markdown
Member

@greptileai

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR corrects spelling and wording in comments, user-facing documentation, add-on metadata, and a historical changelog entry.

  • Corrects three source-code comments without changing executable code.
  • Improves README wording and fixes spelling in manifest descriptions.
  • Preserves YAML structure and all identifiers, APIs, and runtime behavior.

Confidence Score: 5/5

The PR appears safe to merge because it only corrects text and introduces no functional or structural changes.

All changed source lines are comments, and the documentation and YAML edits preserve their existing semantics and valid structure; no actionable issues remain.

Important Files Changed

Filename Overview
Downloads/SidebarItemControl.cs Corrects “contener” to “container” in a comment without affecting behavior.
Games/RomMImport.cs Corrects “incase” to “in case” in a fail-safe comment without changing logic.
README.md Corrects spelling and improves setup instructions while preserving their meaning and link.
RomM.cs Corrects “avaiable” to “available” in a comment without affecting behavior.
manifests/addon.yaml Corrects a spelling error in the user-facing add-on description without altering manifest structure.
manifests/installer.yaml Corrects “pagniated” to “paginated” in a historical changelog entry without altering package metadata.

Reviews (1): Last reviewed commit: "Fix typos in comments, README, and addon..." | Re-trigger Greptile

@gantoine
gantoine merged commit 892c9b7 into rommapp:main Sep 8, 2026
4 checks passed
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