Skip to content

Import cleanup#28

Merged
Arctis-Fireblight merged 2 commits into
Redot-Engine:masterfrom
Tekkitslime:import_cleanup
Jul 10, 2026
Merged

Import cleanup#28
Arctis-Fireblight merged 2 commits into
Redot-Engine:masterfrom
Tekkitslime:import_cleanup

Conversation

@Tekkitslime

@Tekkitslime Tekkitslime commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Fixes #26. Removed leftover Docosaurus import statements :)

Summary by CodeRabbit

  • Documentation
    • Streamlined documentation pages by removing redundant component declarations while preserving existing tabbed content.
    • Corrected and clarified the class-reference guidance to explain that the documentation site does not use Sphinx.
    • Fixed a spelling error in the class-reference documentation.
    • Updated multiple asset library, build, IDE, module, debugging, and documentation pages for improved consistency and maintainability.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 67527421-5294-4fc1-9003-ee35bcf395e0

📥 Commits

Reviewing files that changed from the base of the PR and between 115d67d and 7853e0e.

📒 Files selected for processing (9)
  • Redot-Documentation/docs/Community/asset_library/submitting_to_assetlib.md
  • Redot-Documentation/docs/Contributing/Development/compiling/compiling_for_linuxbsd.md
  • Redot-Documentation/docs/Contributing/Development/compiling/compiling_with_script_encryption_key.md
  • Redot-Documentation/docs/Contributing/Development/compiling/introduction_to_the_buildsystem.md
  • Redot-Documentation/docs/Contributing/Development/configuring_an_ide/visual_studio_code.md
  • Redot-Documentation/docs/Contributing/Development/core_and_modules/custom_modules_in_cpp.md
  • Redot-Documentation/docs/Contributing/Development/debugging/vulkan/vulkan_validation_layers.md
  • Redot-Documentation/docs/Contributing/Documentation/class_reference_primer.md
  • Redot-Documentation/docs/Contributing/Documentation/updating_the_class_reference.md
💤 Files with no reviewable changes (8)
  • Redot-Documentation/docs/Contributing/Development/core_and_modules/custom_modules_in_cpp.md
  • Redot-Documentation/docs/Contributing/Development/debugging/vulkan/vulkan_validation_layers.md
  • Redot-Documentation/docs/Contributing/Development/compiling/compiling_with_script_encryption_key.md
  • Redot-Documentation/docs/Contributing/Development/compiling/compiling_for_linuxbsd.md
  • Redot-Documentation/docs/Contributing/Development/configuring_an_ide/visual_studio_code.md
  • Redot-Documentation/docs/Community/asset_library/submitting_to_assetlib.md
  • Redot-Documentation/docs/Contributing/Documentation/class_reference_primer.md
  • Redot-Documentation/docs/Contributing/Development/compiling/introduction_to_the_buildsystem.md

📝 Walkthrough

Walkthrough

The documentation updates remove obsolete Docusaurus Tabs and TabItem imports from several Markdown pages. The class-reference update corrects the Sphinx notice and states that the existing content is currently inapplicable.

Changes

Documentation cleanup

Layer / File(s) Summary
Remove obsolete component imports
docs/Community/asset_library/submitting_to_assetlib.md, docs/Contributing/Development/compiling/*.md, docs/Contributing/Development/configuring_an_ide/visual_studio_code.md, docs/Contributing/Development/core_and_modules/custom_modules_in_cpp.md, docs/Contributing/Development/debugging/vulkan/vulkan_validation_layers.md, docs/Contributing/Documentation/class_reference_primer.md
Removed top-of-file Docusaurus Tabs and TabItem imports while leaving the remaining document content unchanged.
Correct class-reference notice
docs/Contributing/Documentation/updating_the_class_reference.md
Updated the notice to state that the docs site does not use Sphinx and corrected the “Spinx” misspelling.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The wording fix in updating_the_class_reference.md is unrelated to the import cleanup and appears out of scope. Remove or split the unrelated documentation wording change so the PR stays focused on removing leftover Docusaurus imports.
Title check ❓ Inconclusive The title is too generic and does not identify the Docusaurus import removal in docs. Use a more specific title like 'Remove leftover Docusaurus Tabs/TabItem imports from docs'.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The PR removes the leftover Tabs/TabItem import lines from the affected docs while keeping the tab markup intact.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@Arctis-Fireblight Arctis-Fireblight left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Arctis-Fireblight Arctis-Fireblight merged commit eb6be74 into Redot-Engine:master Jul 10, 2026
3 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.

Remove leftover Docusaurus import statements (Tabs/TabItem) from docs pages

2 participants