Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: regressions from headless plugins refactoring #13337

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

xai
Copy link
Contributor

@xai xai commented Jan 31, 2024

Fixes #13320

Contributed on behalf of STMicroelectronics

What it does

Fixes regressions from headless plugins refactoring (#13320 )

  • fix gaps in Inversify DI configuration in the web worker for frontend plugins
  • restore the onDidDeploy() dorwarding lost in the refactoring for headless plugins

How to test

  1. Start browser example
  2. Check that there are no DI related error messages in the debug console
  3. Install or remove a vscode extension
  4. Verify that the UI updates the button correctly after the action
  5. Repeat with electron example

Follow-ups

Review checklist

Reminder for reviewers

@cdamus
Copy link
Contributor

cdamus commented Jan 31, 2024

Commit 9a7a367 restores the forwarding of onDidDeploy() events to the HostedPluginWatcher that was lost in the headless plugins refactoring.

@xai xai marked this pull request as ready for review January 31, 2024 20:17
@xai xai changed the title fix: resolve dependency injection issues fix: regressions from headless plugins refactoring Jan 31, 2024
- fix gaps and errors in Inversify DI configuration in the web worker
  for frontend plugins
- restore the onDidDeploy() forwarding lost in the refactoring for
  headless plugins

Fixes eclipse-theia#13320

Contributed on behalf of STMicroelectronics

Signed-off-by: Olaf Lessenich <olessenich@eclipsesource.com>
Co-authored-by: Christian W. Damus <cdamus.ext@eclipsesource.com>
Signed-off-by: Christian W. Damus <cdamus.ext@eclipsesource.com>
@JonasHelming
Copy link
Contributor

@msujew This fixes a regression, we want to do a patch release with this. A quick review would be great!

@msujew msujew added the plug-in system issues related to the plug-in system label Feb 1, 2024
Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks for the fix 👍

@JonasHelming JonasHelming merged commit 9a51317 into eclipse-theia:master Feb 1, 2024
12 of 14 checks passed
jfaltermeier pushed a commit that referenced this pull request Feb 6, 2024
- fix gaps and errors in Inversify DI configuration in the web worker
  for frontend plugins
- restore the onDidDeploy() forwarding lost in the refactoring for
  headless plugins

Fixes #13320

Contributed on behalf of STMicroelectronics

Signed-off-by: Olaf Lessenich <olessenich@eclipsesource.com>
Co-authored-by: Christian W. Damus <cdamus.ext@eclipsesource.com>
@jfaltermeier jfaltermeier added this to the 1.47.0 milestone Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plug-in system issues related to the plug-in system
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Extensions View does not update as expected
5 participants