Skip to content

chore: add context to workspace-engine release step#1166

Merged
adityachoudhari26 merged 1 commit into
mainfrom
add-ctx-engine-release
Jun 8, 2026
Merged

chore: add context to workspace-engine release step#1166
adityachoudhari26 merged 1 commit into
mainfrom
add-ctx-engine-release

Conversation

@adityachoudhari26

@adityachoudhari26 adityachoudhari26 commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Improved CI/CD pipeline configuration for build optimization.

Copilot AI review requested due to automatic review settings June 8, 2026 14:23
@adityachoudhari26 adityachoudhari26 merged commit 2ed9af2 into main Jun 8, 2026
2 of 3 checks passed
@adityachoudhari26 adityachoudhari26 deleted the add-ctx-engine-release branch June 8, 2026 14:23
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d8904d9d-495c-4e63-b56f-2d7ab5d9e921

📥 Commits

Reviewing files that changed from the base of the PR and between 905e78c and 5e23af8.

📒 Files selected for processing (1)
  • .github/workflows/release.yaml

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

This PR adds an explicit Docker build context path to the build-workspace-engine job in the release workflow, ensuring the shared image build workflow operates on the correct application directory.

Changes

Docker build context for workspace-engine

Layer / File(s) Summary
Build context specification
.github/workflows/release.yaml
The build-workspace-engine job now explicitly passes context: apps/workspace-engine to the shared build-image workflow alongside existing Dockerfile, platform, and tag inputs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A workflow so neat, now precise in its path,
The build context blooms with methodical math,
One line to declare where the engine shall dwell,
In apps/workspace-engine—configured quite well! 🐰✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-ctx-engine-release

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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the release workflow Docker build configuration for workspace-engine to use a narrower build context.

Changes:

  • Set the Docker build context to apps/workspace-engine for the ctrlplane/workspace-engine image.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 100 to +101
dockerfile: apps/workspace-engine/Dockerfile
context: apps/workspace-engine
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