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

[deb] fix image #448

Merged
merged 1 commit into from
Jan 11, 2021
Merged

[deb] fix image #448

merged 1 commit into from
Jan 11, 2021

Conversation

marcdumais-work
Copy link
Member

@marcdumais-work marcdumais-work commented Dec 4, 2020

Minimal fix for this image. The example app would need to be updated
to support vscode extensions to be useful.

I was thinking that this may be the only example we have of packaging a
browser Theia app, along with the rpm example. So we may consider
updating the app, building a .deb and using it to install the app in DEBIAN
-based containers. (not in this PR though)

Signed-off-by: Marc Dumais marc.dumais@ericsson.com

Minimal fix for this image. The example app would need to be updated
to support vscode extensions to be useful.

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

The minimal changes look good to me 👍
I have a minor comment regarding the node version.

@vince-fugnitto
Copy link
Member

@marcdumais-work fine to merge?

@marcdumais-work
Copy link
Member Author

yes, thanks for the reminder.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants