Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Aug 30, 2025

Important

Add cloud package to Docker build process in Dockerfile.runner and Dockerfile.web.

  • Dockerfile Changes:
    • Add packages/cloud directory creation and package.json copy in Dockerfile.runner and Dockerfile.web.
    • Ensures cloud package is included in the build process for both runner and web environments.

This description was created by Ellipsis for a87f011. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested review from jr and mrubens as code owners August 30, 2025 03:10
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 30, 2025
@cte cte merged commit c7d7ad8 into main Aug 30, 2025
10 checks passed
@cte cte deleted the cte/fix-evals branch August 30, 2025 03:11
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Aug 30, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Aug 30, 2025
@dosubot dosubot bot added the bug Something isn't working label Aug 30, 2025
Copy link
Contributor

@roomote roomote bot left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution! I've reviewed the changes and have some suggestions for improvement. The fix correctly adds the cloud package to both Docker builds, which should resolve the build issues.

packages/ipc \
packages/types
packages/types \
packages/cloud
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this intentional? The line is missing a trailing backslash for consistency with the other lines in this mkdir command. Consider:

Suggested change
packages/cloud
packages/cloud \n```
This would maintain consistency with the formatting pattern used for other directories.

packages/ipc \
packages/telemetry \
packages/types \
packages/cloud \
Copy link
Contributor

Choose a reason for hiding this comment

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

Good addition here. The cloud package is properly included with the trailing backslash, maintaining consistency with the rest of the mkdir command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants