Skip to content

Remove duplicated logic to unify code paths#48291

Merged
KyleBarton merged 1 commit intomainfrom
fix_in_progress_message_for_devcontainer_quick_action
Feb 3, 2026
Merged

Remove duplicated logic to unify code paths#48291
KyleBarton merged 1 commit intomainfrom
fix_in_progress_message_for_devcontainer_quick_action

Conversation

@KyleBarton
Copy link
Collaborator

Small fix for a bug introduced in #47411
In-progress dev container creation didn't show up in modal because of a duplicated code path. This unifies the logic and ensures that "creating dev container" shows up while creation in progress.

Release Notes:

  • Fixed modal for creating dev container

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 3, 2026
@KyleBarton KyleBarton enabled auto-merge (squash) February 3, 2026 19:26
@KyleBarton KyleBarton merged commit f07305b into main Feb 3, 2026
29 checks passed
@KyleBarton KyleBarton deleted the fix_in_progress_message_for_devcontainer_quick_action branch February 3, 2026 19:36
adb-sh pushed a commit to adb-sh/zed that referenced this pull request Feb 5, 2026
Small fix for a bug introduced in zed-industries#47411
In-progress dev container creation didn't show up in modal because of a
duplicated code path. This unifies the logic and ensures that "creating
dev container" shows up while creation in progress.

Release Notes:

- Fixed modal for creating dev container
rtfeldman pushed a commit that referenced this pull request Feb 5, 2026
Small fix for a bug introduced in #47411
In-progress dev container creation didn't show up in modal because of a
duplicated code path. This unifies the logic and ensures that "creating
dev container" shows up while creation in progress.

Release Notes:

- Fixed modal for creating dev container
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.

1 participant