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

[yangster-docker@next, theia-full-next] not working #294

Closed
marcdumais-work opened this issue Feb 4, 2020 · 1 comment · Fixed by #295
Closed

[yangster-docker@next, theia-full-next] not working #294

marcdumais-work opened this issue Feb 4, 2020 · 1 comment · Fixed by #295
Labels
bug ci full

Comments

@marcdumais-work
Copy link
Member

It builds but the app can't start

Client-side error:

index.js:95 Error: Cannot apply @injectable decorator multiple times.
    at injectable.js:8
    at Reflect.js:540
    at Object.<anonymous> (Reflect.js:130)
    at a (decorator_utils.js:42)
    at Object.t.decorate (decorator_utils.js:55)
    at Object.<anonymous> (widget.ts:25)
    at n (bootstrap:19)
    at Object.<anonymous> (index.ts:17)
    at n (bootstrap:19)
    at Object.<anonymous> (dialogs.ts:20)

This is caused by the fact that the app pulls 2 versions of @theia/core. The root cause looks to be the theia-yang-extension@next extension:

theia-ide/yangster#85
@marcdumais-work marcdumais-work changed the title [yangster-docker@next] not working [yangster-docker@next, theia-full] not working Feb 4, 2020
@marcdumais-work marcdumais-work changed the title [yangster-docker@next, theia-full] not working [yangster-docker@next, theia-full-next] not working Feb 4, 2020
@marcdumais-work marcdumais-work mentioned this issue Feb 4, 2020
Merged
1 task
@marcdumais-work
Copy link
Member Author

Way forward: after discussion, we'll remove the yangster image (latest and next) from CI and remove theia-yang-extension from the theia-full:next image. See theia-ide/yangster#85 (comment)

marcdumais-work added a commit that referenced this issue Feb 6, 2020
@marcdumais-work
[yangster][full] fix CI
6c4a2d6 
Due to limited resources the yangster-related extensions may not keep-up
with the latest Theia platform at all times. To minimize CI issues here,
this commit removes the theia-yangster image (both latest and next) from
CI, and also removes theia-yangs-extension from theia-full:next.

Fixes #294

Signed-off-by: Marc Dumais <[email protected]>
@marcdumais-work marcdumais-work mentioned this issue Feb 6, 2020
Merged
marcdumais-work added a commit that referenced this issue Feb 6, 2020
@marcdumais-work
[yangster][full] fix CI
9139e67 
Due to limited resources the yangster-related extensions may not keep-up
with the latest Theia platform at all times. To minimize CI issues here,
this commit removes the theia-yangster image (both latest and next) from
CI, and also removes theia-yangs-extension from theia-full:next.

Fixes #294

Signed-off-by: Marc Dumais <[email protected]>
kbarbounakis pushed a commit to kbarbounakis/theia-apps that referenced this issue Feb 21, 2020
@marcdumais-work @kbarbounakis
[yangster][full] fix CI
ab7f4c4 
Due to limited resources the yangster-related extensions may not keep-up
with the latest Theia platform at all times. To minimize CI issues here,
this commit removes the theia-yangster image (both latest and next) from
CI, and also removes theia-yangs-extension from theia-full:next.

Fixes theia-ide#294

Signed-off-by: Marc Dumais <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug ci full
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[yangster][full] fix CI theia-ide/theia-apps
1 participant
@marcdumais-work