-
-
Notifications
You must be signed in to change notification settings - Fork 97
Open
Milestone
Description
We see several demands from users to selectively deactivate features of this extension. This is either because of performance reasons (long running queries in the background) or simply because features are not used and hence dependencies shouldn't be required (e.g. buildifier).
Features that qualify for deactivation:
- Target Tree in the explorer view
- CodeLens
- Bazel Commands requiring bazel (build/test/queries)
- Syntax Highlighting (? is this really something that you would want to turn off)
- Language Server
- Buildifier Linting
I assume there are some pitfalls to avoid here:
- Deactivating bazel commands will also render almost any other functionality useless
- We already have some settings that implicitly deactivate some features when the executable path is empty (language server). We might need to introduce a breaking change in order to provide a consistent user interface for configuration
- A lot of the setup code for the features is done during activation of the extension. It probably requires quite some refactoring to be able to deactivate features without requiring a reload of the complete IDE.
For reference, I saw demands of this here (and some of these could be closed if this issue is solved):
- feat(config): add an option to fully disable queries #451
- Improve logging capabilities of the extension #227
- don't throw when buildifier is not found #193
- make buildifier warning less aggressive #402
- Make test explorer functionality optional #403
- Want a config to toggle buildifier lint diagnostics #449
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels