Skip to content
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.

languages: drop '@theia/languages' #392

Merged
merged 1 commit into from
Aug 7, 2020
Merged

Conversation

vince-fugnitto
Copy link
Member

What it does

Fixes: #387

The following pull-request removes the deprecated @theia/languages extension from both images and apps in favor of using vscode extensions to provide language-support.

How to test

Verify that CI passes successfully.

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com

The following commit removes the `@theia/languages` extension dependency
since it is deprecated in favor of using vscode extensions to provide
language-support.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
@marcdumais-work
Copy link
Member

I think we'll need to merge #393 before the C/C++ related images can build correctly. We have a circular dependency - we'll need to merge one or the other on faith :)

Copy link
Member

@marcdumais-work marcdumais-work left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @vince-fugnitto

@vince-fugnitto vince-fugnitto merged commit ac374a5 into master Aug 7, 2020
@vince-fugnitto vince-fugnitto deleted the vf/languages-drop branch August 7, 2020 16:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dependency: remove '@theia/languages' extension
2 participants