Skip to content

Fix dockerfile image and alias parsing#53538

Merged
KyleBarton merged 3 commits intomainfrom
fix_dockerfile_image_and_alias_parsing
Apr 9, 2026
Merged

Fix dockerfile image and alias parsing#53538
KyleBarton merged 3 commits intomainfrom
fix_dockerfile_image_and_alias_parsing

Conversation

@KyleBarton
Copy link
Copy Markdown
Collaborator

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Closes #52928

Release Notes:

  • Fixed handling of multi-stage and stage-specified dockerfiles in dev container manifests
  • Fixed the way we find the base image in a dev container when build args need expansion
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 9, 2026
@zed-codeowner-coordinator zed-codeowner-coordinator Bot requested review from a team, SomeoneToIgnore and reflectronic and removed request for a team April 9, 2026 15:52
@KyleBarton KyleBarton merged commit bd50418 into main Apr 9, 2026
31 checks passed
@KyleBarton KyleBarton deleted the fix_dockerfile_image_and_alias_parsing branch April 9, 2026 20:27
piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [x] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Closes zed-industries#52928

Release Notes:

- Fixed handling of multi-stage and stage-specified dockerfiles in dev
container manifests
- Fixed the way we find the base image in a dev container when build
args need expansion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

3 participants