Skip to content

workspace: Delete shared_screen module - #42

Merged
jasonsmithio merged 2 commits into
mainfrom
drop-shared-screen
May 24, 2026
Merged

workspace: Delete shared_screen module#42
jasonsmithio merged 2 commits into
mainfrom
drop-shared-screen

Conversation

@jasonsmithio

Copy link
Copy Markdown
Collaborator

Summary

  • Remove pub mod shared_screen and pub use shared_screen::SharedScreen from workspace.rs
  • Delete crates/workspace/src/shared_screen.rs (126 lines)

The last consumer (create_shared_screen on the AnyActiveCall trait) was removed in PR #41. No crate outside collab_ui/collab/call referenced SharedScreen, and call's impl was already deleted.

Test plan

  • cargo check -p workspace -p paddleboard clean
  • ./script/clippy -p workspace -p paddleboard clean

Release Notes:

  • N/A

jasonsmithio and others added 2 commits May 24, 2026 09:03
The last consumer (create_shared_screen on AnyActiveCall) was removed
in the previous PR. Drop the module declaration, re-export, and file.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions

github-actions Bot commented May 24, 2026

Copy link
Copy Markdown

Upstream drift report

Comparing PaddleBoard HEAD (accd2b9049) against zed-industries/zed:main (eb2223c080).
Last common ancestor: 8ca194d833.

Crate Files Insertions Deletions // PaddleBoard: tags
crates/agent 26 +25357 -1407 0
crates/agent_ui 33 +4477 -3836 6
crates/buffer_diff 1 +0 -4 0
crates/client 5 +116 -316 2
crates/collab 6 +20 -60 0
crates/collab_ui 2 +2 -2 0
crates/editor 15 +347 -542 0
crates/gpui 6 +60 -447 0
crates/language 2 +4 -2 0
crates/language_core 0 +0 -0 0
crates/lsp 0 +0 -0 0
crates/multi_buffer 0 +0 -0 0
crates/project 20 +1309 -689 1
crates/rope 0 +0 -0 0
crates/settings_content 6 +197 -31 6
crates/text 0 +0 -0 0
crates/workspace 13 +176 -783 1
Total 135 +32065 -8119 16

Generated by script/check-upstream-drift. See FORK_HYGIENE.md for context.

@jasonsmithio
jasonsmithio merged commit c9388ac into main May 24, 2026
1 check passed
jasonsmithio added a commit that referenced this pull request May 31, 2026
workspace: Delete shared_screen module
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