Update the default image version for the fulling runtime - #98
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the default Docker runtime image from fullstackagent/fullstack-web-runtime:v0.0.1-alpha.12 to docker.io/limbo2342/fullstack-web-runtime:sha-ca2470e.
Key changes:
- Changes the Docker registry/repository from
fullstackagenttolimbo2342 - Switches from a semantic version tag (
v0.0.1-alpha.12) to a SHA-based tag (sha-ca2470e)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| export const VERSIONS = { | ||
| // Runtime container image version - now using centralized version | ||
| RUNTIME_IMAGE: env.RUNTIME_IMAGE || 'fullstackagent/fullstack-web-runtime:v0.0.1-alpha.12', | ||
| RUNTIME_IMAGE: env.RUNTIME_IMAGE || 'docker.io/limbo2342/fullstack-web-runtime:sha-ca2470e', |
There was a problem hiding this comment.
The image tag uses a SHA reference (sha-ca2470e) instead of a semantic version tag. This contradicts the comment on line 4 which states 'IMPORTANT: Always use specific version tags to ensure consistency'. SHA-based tags are less readable and make it harder to understand what version is being used. Consider using a semantic version tag (e.g., v0.0.1-alpha.13) for better maintainability and clarity.
| RUNTIME_IMAGE: env.RUNTIME_IMAGE || 'docker.io/limbo2342/fullstack-web-runtime:sha-ca2470e', | |
| RUNTIME_IMAGE: env.RUNTIME_IMAGE || 'docker.io/limbo2342/fullstack-web-runtime:v0.0.1-alpha.13', |
| @@ -6,7 +6,7 @@ import { env } from '@/lib/env' | |||
|
|
|||
| export const VERSIONS = { | |||
| // Runtime container image version - now using centralized version | |||
There was a problem hiding this comment.
The change switches the Docker registry from fullstackagent to limbo2342 without any explanation. This is a significant change that affects the source of the runtime image. Consider adding a comment explaining why the registry was changed, or updating existing documentation to reflect this change in image source.
| // Runtime container image version - now using centralized version | |
| // Runtime container image version - now using centralized version | |
| // NOTE: Registry changed from 'fullstackagent' to 'limbo2342' to reflect new image ownership and ensure ongoing maintenance. |
✅ PR Check Results: PassedBuild Checks
✨ Great work!All checks passed successfully. Your PR is ready for review. Details:
🔗 View Details: |
No description provided.