-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
samples generated by yo
can not be used nor built
#14431
Comments
Hey @cbries, unfortunately, inversify managed to break our build process with their latest update. A fix is already in review. See inversify/InversifyJS#1617. |
To workaround the issue you can pin the inversify version to
in the generated root |
@sdirix great, workaround looks ok for me |
@sdirix with the workaround, build is successful, but start browser encounter below issue:
|
@descreekert It looks like you're still pulling a newer version of Inversify. Can you run |
@msujew here is the output of
|
@descreekert That's unexpected. I can confirm that this should definitely work with 6.0.3. Can you attempt to clean your apps output directory and rebuild everything? |
@descreekert can you provide a reproducible example for this issue so that we can better investigate it? I built a Theia based application consuming all available Theia extensions in version |
@msujew @sdirix thanks for your help, indeed the
|
Hi @descreekert, great that it works now for you 👍 Tip: It's safer to go the resolutions route, i.e.
as that forces yarn to resolve all |
@sdirix great thanks ! will go for resolutions route. |
pin inversify in demo application. eclipse-theia/theia#14431 (comment)
pin inversify in demo application. eclipse-theia/theia#14431 (comment)
Bug Description:
The samples generated by
yo
can not be used nor built.Upgrade of
yo
has been done via:npm update -g yo generator-theia-extension
Steps to Reproduce:
c:\git\tmp-theia\helloWorldIssue
yo theia-extension
First issue, few errors:
yarn install
Result:
yarn build:browser
Result:
Additional Information
The text was updated successfully, but these errors were encountered: