Skip to content

Disable some Compose features if Docker's extension is installed #75

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rcjsuen
Copy link

@rcjsuen rcjsuen commented May 9, 2025

Docker's extension bundles a language server that provides some Compose features that overlap with the ones from the Container Tools extension. Red Hat's YAML extension also does something similar which was previously accounted for. The code has now been changed to disable select Compose features if Docker's extension or Red Hat's YAML extension is installed.

To test this, one should install specific builds of Docker's extension that are being generated by docker/vscode-extension#108.

Docker's extension bundles a language server that provides some Compose
features that overlap with the ones from the Container Tools extension.
Red Hat's YAML extension also does something similar which was
previously accounted for. The code has now been changed to disable
select Compose features if Docker's extension or Red Hat's YAML
extension is installed.

Signed-off-by: Remy Suen <[email protected]>
@rcjsuen rcjsuen requested a review from a team as a code owner May 9, 2025 11:27
@rcjsuen rcjsuen marked this pull request as draft May 9, 2025 11:27
@rcjsuen
Copy link
Author

rcjsuen commented May 9, 2025

@bwateratmsft I've converted this to a draft for now. Will switch it back after we've figured out the coordination of things to ensure things are generally smooth and non-disruptive.

@bwateratmsft
Copy link
Collaborator

Sounds good. Code looks great, thanks @rcjsuen!

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.

2 participants