Skip to content

chore: abort extension host given type errors

Mark Florian requested to merge fail-run-extension-on-type-errors into main

Description

chore: abort extension host given type errors

Before this commit, if you launch "Run Extension" when there are type errors in the project, the extension host would start, but the extension won't be loaded because the watch:desktop script exited due to the failure.

This is confusing, because it requires digging into the logs to see what happened.

Now, that failure is caught and presented directly, and the extension host is not started.

It could be useful to ignore type errors in such cases while developing the extension, but for now this at least clarifies why the extension isn't loaded.

Related Issues

Related to #1482 (closed)

How has this been tested?

  1. Introduce a type error (e.g., something that disagress with the type of an import from @gitlab-org/gitlab-lsp.
  2. Run the extension (F5)
  3. Note that the extension development host fails to start, with a modal prompt.

Screenshots (if appropriate)

What CHANGELOG entry will this MR create?

  • fix: Bug fix fixes - a user-facing issue in production - included in changelog
  • feature: New feature - a user-facing change which adds functionality - included in changelog
  • BREAKING CHANGE: (fix or feature that would cause existing functionality to change) - should bump major version, mentioned in the changelog
  • None - other non-user-facing changes
Edited by Mark Florian

Merge request reports

Loading