Skip to content

cleanup Workflow/ folder#29

Draft
Tekkitslime wants to merge 3 commits into
Redot-Engine:masterfrom
Tekkitslime:cleanup
Draft

cleanup Workflow/ folder#29
Tekkitslime wants to merge 3 commits into
Redot-Engine:masterfrom
Tekkitslime:cleanup

Conversation

@Tekkitslime

Copy link
Copy Markdown
Contributor

changed non existing links, fixed image paths, made note blocks more consistent.

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f86f4642-fa7c-469f-aaaa-d861ac735eab

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ 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.

Comment on lines +32 to +36
whether the issue is a regression in 26.2 or not.

- If the issue **is present** in 3.x, then you'll need to check whether the issue
occurs in older 3.x versions as well.
- If the issue is **not present** in 3.x, then you can try older 4.0 alphas and
- If the issue **is present** in 26.x, then you'll need to check whether the issue
occurs in older 26.x versions as well.
- If the issue is **not present** in 26.x, then you can try older 26.2 alphas and

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.

Probably shouldnt reference 26.2 because that will get very dated very fast. I'd prolly keep it as 26.x

Comment on lines 31 to 33
To get the Redot source code for compiling, see [doc_getting_source](getting_source.md).

For a general overview of SCons usage for Redot, see [doc_introduction_to_the_buildsystem](introduction_to_the_buildsystem.md).

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.

The .md file extension needs to be removed from these, and they need to be prefixed with doc_

Comment on lines +110 to +111
<!-- TODO(Tekk): class_reference.md doesnt exist -->
<!-- TODO(Tekk): does this need to be re-written for current site? -->

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.

The class reference will work a bit differently.

These docs can typically be referenced with the class_ prefix for now, but won't actually do anything till we have the ClassDB implemented.

The idea, is that the site will pull the latest xml files from the engine repo for each versioned branch every 24 hours, and then use those to generate the class reference pages instead of using scripts to convert them to RST or Markdown like previous iterations did.

I think in the case of doc_class_reference I believe it was dynamically generated and updated by the previously mentioned conversion scripts.

- Open the pull request page. Click the *fork*'s branch name near the top of the page:

![Image](img/testing_pull_requests_access_fork.png)
![Image](img/Contributing/Workflow/testing_pull_requests_access_fork.png)

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.

Image links should be prefixed with a forward slash, so this should be:

Suggested change
![Image](img/Contributing/Workflow/testing_pull_requests_access_fork.png)
![Image](/img/Contributing/Workflow/testing_pull_requests_access_fork.png)

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