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

cli: support downloading .theia plugins #10082

Merged
merged 1 commit into from
Sep 13, 2021
Merged

Conversation

vince-fugnitto
Copy link
Member

What it does

Fixes: #9264.

The pull-request updates the download-plugins script to allow the downloading of .theia files (theia-plugins).
Previously, the operation would fail as it was treated as an unsupported file type.

How to test

  1. include the following entry under theiaPlugins:
"theia-helloworld": "https://github.com/vince-fugnitto/theia-plugin-helloworld/releases/download/0.0.1/hello.theia"
  1. perform yarn download:plugins
  2. confirm the download is successful, and the resource is present under the plugins/ folder
  3. start the application
  4. execute the command theia: hello world and confirm it is successul

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto [email protected]

The commit updates the `download-plugins` script to allow the
downloading of `.theia` (theia-plugins). Previously, the download would
fail if an application references a `.theia` file as it was an
unsupported file extension.

Signed-off-by: vince-fugnitto <[email protected]>
@vince-fugnitto vince-fugnitto added the theia-cli issues related to the theia-cli label Sep 10, 2021
@vince-fugnitto vince-fugnitto self-assigned this Sep 10, 2021
@vince-fugnitto vince-fugnitto merged commit 6401485 into master Sep 13, 2021
@vince-fugnitto vince-fugnitto deleted the vf/download-theia branch September 13, 2021 12:51
@github-actions github-actions bot added this to the 1.18.0 milestone Sep 13, 2021
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.

cli: update 'download:plugins' script to support '.theia' plugins
2 participants