Skip to content

[@theia/console] Remove workaround that is no longer necessary#8741

Merged
paul-marechal merged 1 commit intoeclipse-theia:masterfrom
RDIL:master
Nov 12, 2020
Merged

[@theia/console] Remove workaround that is no longer necessary#8741
paul-marechal merged 1 commit intoeclipse-theia:masterfrom
RDIL:master

Conversation

@RDIL
Copy link
Contributor

@RDIL RDIL commented Nov 11, 2020

Signed-off-by: Reece Dunham me@rdil.rocks

What it does

My PR in Anser (IonicaBizau/anser#60) landed, meaning that the misleading TypeScript types no longer exist, so we can just use the proper way of importing Anser.

How to test

I've verified that the Theia console still works.

Review checklist

Reminder for reviewers

@vince-fugnitto vince-fugnitto added dependencies issues that plan to update dependencies quality issues related to code and application quality labels Nov 11, 2020
Signed-off-by: Reece Dunham <me@rdil.rocks>
Copy link
Member

@paul-marechal paul-marechal left a comment

Choose a reason for hiding this comment

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

LGTM, CI failure seems unrelated.

@paul-marechal paul-marechal merged commit 4974529 into eclipse-theia:master Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies issues that plan to update dependencies quality issues related to code and application quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants