Skip to content

Fix various typos listed in godotengine#91521#1206

Open
AR-DEV-1 wants to merge 1 commit intoRedot-Engine:masterfrom
AR-DEV-1:various-typo-fixes
Open

Fix various typos listed in godotengine#91521#1206
AR-DEV-1 wants to merge 1 commit intoRedot-Engine:masterfrom
AR-DEV-1:various-typo-fixes

Conversation

@AR-DEV-1
Copy link
Contributor

@AR-DEV-1 AR-DEV-1 commented Mar 4, 2026

TL;DR

  • Fixes all of the currently listed typos & UI paper cuts in godotengine#91521 (except for godotengine#116481)
  • Related to our codestyle

Note

Contributed by 2LazyDevs

Summary by CodeRabbit

  • Bug Fixes

    • Corrected capitalization of "Physical Location" label in settings UI
  • Documentation

    • Fixed grammatical errors and improved clarity in class documentation
    • Enhanced property descriptions with corrected terminology and behavioral notes
    • Updated method documentation for accuracy

@AR-DEV-1
Copy link
Contributor Author

AR-DEV-1 commented Mar 4, 2026

Godot version: godotengine/godot#116543

@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f4c46e7e-09cd-4582-b022-ea87a9b5258d

📥 Commits

Reviewing files that changed from the base of the PR and between 3ef8058 and 194988f.

📒 Files selected for processing (5)
  • core/string/ustring.cpp
  • doc/classes/FoldableContainer.xml
  • doc/classes/Node.xml
  • doc/classes/Resource.xml
  • editor/settings/input_event_configuration_dialog.cpp

Walkthrough

This PR contains minor fixes and documentation updates across five files. Changes include a typo correction in source code comments, grammar fixes and accuracy improvements in class documentation, and UI text capitalization standardization. No functional or behavioral changes are introduced.

Changes

Cohort / File(s) Summary
Source Code Comment Fix
core/string/ustring.cpp
Corrected typo in wchar_t comparison helper comments: "32-bi" → "32-bit".
Class Documentation Updates
doc/classes/FoldableContainer.xml, doc/classes/Node.xml, doc/classes/Resource.xml
Grammar and accuracy fixes: "will becomes folded" → "will become folded", "ancestors" → "descendants" in find_children documentation, and improved phrasing of resource_scene_unique_id documentation.
UI Text Capitalization
editor/settings/input_event_configuration_dialog.cpp
Standardized UI label capitalization: "Physical location" → "Physical Location" in both label and accessibility name.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix various typos listed in godotengine#91521' directly and clearly summarizes the main change—correcting multiple typos identified in the referenced issue.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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.

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