Skip to content
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

ci(compile-references): add error info for configureTypeScriptNavigation #9667

Conversation

zWingz
Copy link
Contributor

@zWingz zWingz commented Jun 30, 2021

What it does

add error info in configureTypeScriptNavigation. fixed #9666

How to test

  • remove a property in compilerOptions.paths
  • yarn prepare:references

image

Review checklist

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.

I confirmed that the new error displays correctly, thank you 👍

$ yarn prepare:references
yarn run v1.22.4
$ node scripts/compile-references.js
error: @theia/monorepo invalid mapped path: {"@theia/callhierarchy/lib/*": "undefined"}
error: @theia/monorepo invalid mapped path: {"@theia/console/lib/*": "undefined"}
error: @theia/monorepo invalid mapped path: {"@theia/core/lib/*": "undefined"}
error: @theia/monorepo invalid mapped path: {"@theia/debug/lib/*": "undefined"}
error: @theia/monorepo invalid mapped path: {"@theia/editor-preview/lib/*": "undefined"}
error: @theia/monorepo invalid mapped path: {"@theia/editor/lib/*": "undefined"}
info: /home/evinfug/workspaces/theia/tsconfig.json updated.

@vince-fugnitto vince-fugnitto added the theia-cli issues related to the theia-cli label Jun 30, 2021
@vince-fugnitto vince-fugnitto merged commit 74cc16e into eclipse-theia:master Jun 30, 2021
@vince-fugnitto vince-fugnitto added this to the 1.15.0 milestone Jun 30, 2021
@zWingz zWingz deleted the bugfix/add-error-info-in-compile-reference branch July 1, 2021 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theia-cli issues related to the theia-cli
Projects
None yet
Development

Successfully merging this pull request may close these issues.

compile-references miss error info in configureTypeScriptNavigation
2 participants