Skip to content

Update release CI scripts - #1160

Merged
Arctis-Fireblight merged 1 commit into
Redot-Engine:devfrom
Arctis-Fireblight:update-build-scripts
Jan 18, 2026
Merged

Arctis-Fireblight merged 1 commit into
Redot-Engine:devfrom
Arctis-Fireblight:update-build-scripts

Conversation

@Arctis-Fireblight

@Arctis-Fireblight Arctis-Fireblight commented Jan 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Enhanced CI/CD pipeline with automatic cleanup of build artifacts and refined conditional release execution to ensure more reliable and controlled release builds.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Jan 18, 2026 •

Copy link
Copy Markdown

Walkthrough

This change modifies the GitLab CI configuration to add cleanup steps that remove build artifacts (bin/obj directory) after build stages and converts job execution conditions from unconditional to release-gated using conditional rules ($DO_RELEASE == "true") for specific Linux editor build jobs.

Changes

Cohort / File(s) Summary
CI/CD Pipeline Configuration
.gitlab-ci.yml
Added cleanup steps removing build artifacts (rm -rf bin/obj) across multiple stages; changed job execution rules from when: always to conditional if: $DO_RELEASE == "true" for linux-editor-x64-mono and linux-editor-x64 jobs; minor formatting adjustments to version logic blocks with no functional alteration to version string generation

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

topic:buildsystem

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update release CI scripts' accurately describes the main change—modifications to the CI/CD pipeline in .gitlab-ci.yml for release builds, including cleanup steps and conditional execution logic.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


Comment @coderabbitai help to get the list of available commands and usage tips.

@Arctis-Fireblight
Arctis-Fireblight merged commit e4ea6b0 into Redot-Engine:dev Jan 18, 2026
17 checks passed
@Arctis-Fireblight
Arctis-Fireblight deleted the update-build-scripts branch January 30, 2026 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant