Skip to content

fix: clean up @workspace dead code after removing from context menu#2670

Merged
chungjac merged 4 commits intoaws:feature/removing-at-workspacefrom
chungjac:remove-workspace-command
Apr 2, 2026
Merged

fix: clean up @workspace dead code after removing from context menu#2670
chungjac merged 4 commits intoaws:feature/removing-at-workspacefrom
chungjac:remove-workspace-command

Conversation

@chungjac
Copy link
Copy Markdown
Contributor

Previous PR which removed the @workspace functionality: #2669

Problem

After removing @workspace from the context commands menu, there is dead code remaining.

Solution

Clean up dead code

@chungjac chungjac force-pushed the remove-workspace-command branch from 77924b6 to ceb3637 Compare March 18, 2026 22:00
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.46%. Comparing base (bf7c32f) to head (3ac5b00).
⚠️ Report is 2 commits behind head on feature/removing-at-workspace.

Additional details and impacted files
@@                        Coverage Diff                        @@
##           feature/removing-at-workspace    #2670      +/-   ##
=================================================================
- Coverage                          60.51%   60.46%   -0.06%     
=================================================================
  Files                                279      279              
  Lines                              66299    66189     -110     
  Branches                            4257     4243      -14     
=================================================================
- Hits                               40121    40021     -100     
+ Misses                             26093    26084       -9     
+ Partials                              85       84       -1     
Flag Coverage Δ
unittests 60.46% <100.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@chungjac chungjac changed the base branch from main to feature/removing-at-workspace April 2, 2026 19:39
@chungjac chungjac merged commit f641dc9 into aws:feature/removing-at-workspace Apr 2, 2026
5 checks passed
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.

6 participants