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

download:plugins fails if "theiaPlugins" is not present in package.json #8053

Closed
lal12 opened this issue Jun 19, 2020 · 3 comments · Fixed by #8058
Closed

download:plugins fails if "theiaPlugins" is not present in package.json #8053

lal12 opened this issue Jun 19, 2020 · 3 comments · Fixed by #8058
Labels
documentation issues related to documentation theia-cli issues related to the theia-cli 🤔 needs more info issues that require more info from the author

Comments

@lal12
Copy link

lal12 commented Jun 19, 2020

Bug Description:

download:plugins fails if "theiaPlugins" is not present in package.json

Steps to Reproduce:

  1. Follow guide on https://theia-ide.org/docs/composing_applications/
  2. Remove "theiaPlugins" key from package.json
  3. Run "theia download:plugins" or just "yarn"

Additional Information

  • Operating System: Ubuntu 20.04
  • Theia Version: 1.2.0 and next
@akosyakov
Copy link
Member

What would you expect? Docs says that you should have theiaPlugins.

@akosyakov akosyakov added 🤔 needs more info issues that require more info from the author documentation issues related to documentation theia-cli issues related to the theia-cli labels Jun 19, 2020
@vince-fugnitto
Copy link
Member

@lal12 the download:plugins script should only be used if you are actually planning to download the plugins defined in the package.json. If this is not your intentions then you should not call it (and neither should your prepare script). There is no reason to call the script if this is not your goal.

@lal12
Copy link
Author

lal12 commented Jun 19, 2020

I would assume that you could have no object in there, if you are using a bunch of standard scripts maybe from another project. I expected a npm/yarn compatible behavior, which also won't fail if no dependency property is specified.

However I also get your position and think failing is an acceptable option. But in this case I think there should be a proper error message, currently there is just a TypeError.

vince-fugnitto added a commit that referenced this issue Jun 19, 2020
Fixes: #8053

The following commit updates the `download:plugins` script to report
when the mandatory `theiaPlugins` property is missing from the
`package.json` resulting in no plugins being downloaded.

Signed-off-by: vince-fugnitto <[email protected]>
vince-fugnitto added a commit that referenced this issue Jun 19, 2020
Fixes: #8053

The following commit updates the `download:plugins` script to report
when the mandatory `theiaPlugins` property is missing from the
`package.json` resulting in no plugins being downloaded.

Signed-off-by: vince-fugnitto <[email protected]>
vince-fugnitto added a commit that referenced this issue Jun 19, 2020
Fixes: #8053

The following commit updates the `download:plugins` script to report
when the mandatory `theiaPlugins` property is missing from the
`package.json` resulting in no plugins being downloaded.

Signed-off-by: vince-fugnitto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation issues related to documentation theia-cli issues related to the theia-cli 🤔 needs more info issues that require more info from the author
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants