Skip to content

revert: one image per devcontainer.json (#60) - feature abandoned - #66

Merged
alexanderilyin merged 2 commits into
mainfrom
revert/multi-config-builds
Oct 7, 2026
Merged

alexanderilyin merged 2 commits into
mainfrom
revert/multi-config-builds

Conversation

@alexanderilyin

Copy link
Copy Markdown
Member

Reverts #60 before 1.0, so images[], instances and dryRun never ship in a release and aren't part of the 1.x API promise. Plans 004 and 005 record the feature as abandoned.

Why: Coder names an agent after its Terraform resource name and fails with duplicate agent name otherwise, so the template couldn't give each image its own agent. Without that, the multiple-image API has no consumer.

After this, a repository whose only configs are in .devcontainer/<folder>/ behaves as in 0.5.0: it falls back to fallbackImage, or fails.

🤖 Generated with Claude Code

alexanderilyin and others added 2 commits October 7, 2026 12:53
…-builds"

This reverts commit a4ddb21, reversing
changes made to c0b0951.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b8128380-c484-4dc0-9852-089d91a9cb46
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@alexanderilyin
alexanderilyin merged commit a982014 into main Oct 7, 2026
10 checks passed
@alexanderilyin
alexanderilyin deleted the revert/multi-config-builds branch October 7, 2026 19:56
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.

1 participant