Skip to content

dev_container: Fix environment variables without an equals sign being treated as fatal parsing errors#53864

Merged
Veykril merged 1 commit intozed-industries:mainfrom
pigletfly:fix-dev-container-env
Apr 16, 2026
Merged

dev_container: Fix environment variables without an equals sign being treated as fatal parsing errors#53864
Veykril merged 1 commit intozed-industries:mainfrom
pigletfly:fix-dev-container-env

Conversation

@pigletfly
Copy link
Copy Markdown
Contributor

@pigletfly pigletfly commented Apr 14, 2026

in vscode the environment variables without an equals sign are ignored https://github.com/devcontainers/cli/blob/main/src/spec-node/utils.ts#L488-L498
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 #53470

Release Notes:

  • dev_container: Fix environment variables without an equals sign being treated as fatal parsing errors.

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 14, 2026
@zed-codeowner-coordinator zed-codeowner-coordinator Bot requested review from a team, Veykril and reflectronic and removed request for a team April 14, 2026 05:50
Copy link
Copy Markdown
Member

@Veykril Veykril left a comment

Choose a reason for hiding this comment

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

Thanks!

@Veykril Veykril enabled auto-merge (squash) April 14, 2026 08:08
@maxdeviant maxdeviant changed the title dev_container: Fix environment variables without an equals sign were treated as fatal parsing errors. dev_container: Fix environment variables without an equals sign being treated as fatal parsing errors Apr 14, 2026
@Veykril Veykril merged commit 54f5444 into zed-industries:main Apr 16, 2026
53 of 56 checks passed
@pigletfly pigletfly deleted the fix-dev-container-env branch April 18, 2026 06:39
G36maid pushed a commit to G36maid/zed that referenced this pull request Apr 29, 2026
…treated as fatal parsing errors. (zed-industries#53864)

in vscode the environment variables without an equals sign are ignored
https://github.com/devcontainers/cli/blob/main/src/spec-node/utils.ts#L488-L498
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#53470

Release Notes:

- Fix environment variables without an equals sign were treated as fatal
parsing errors.
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to parse environment variable

3 participants