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

make extension deploy command extensible #8919

Merged
merged 1 commit into from
Jan 14, 2021

Conversation

odeliat
Copy link
Contributor

@odeliat odeliat commented Jan 5, 2021

Signed-off-by: odelia odelia.zehavi@sap.com

What it does

Make the extension deploy from command pallet extensible

How to test

  • Click cmd+p
  • Enter > Plugin: Deploy Plugin by Id
  • Enter vscode:extension/hollowtree.vue-snippets
  • Check that the extension is installed

In this branch you can see the ability in work:
https://github.com/odeliat/theia/tree/extensible-deploy-cmd-test

  • Run the branch
  • Install the extension as described above
  • See in the console the massage - activate injectable method

Review checklist

Reminder for reviewers

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.

@odeliat thank you for updating the pull-request, I have a minor comment regarding the changelog entry.

When ready with the updates, please be sure to squash your commits (into a single commit), and include a proper sign-off before it can be accepted.

CHANGELOG.md Outdated Show resolved Hide resolved
…em` in order to make extension deploy command extensible

Signed-off-by: odelia <odelia.zehavi@sap.com>
@odeliat odeliat force-pushed the extesible-deploy-cmd branch from 0333612 to d3f2817 Compare January 6, 2021 13:50
Copy link
Member

@amiramw amiramw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM and tested deploy by id command to make sure there are no regressions.

@vince-fugnitto vince-fugnitto added extensibility issues to simplify ability to extend Theia plug-in system issues related to the plug-in system labels Jan 13, 2021
@amiramw amiramw merged commit 31d4749 into eclipse-theia:master Jan 14, 2021
@odeliat odeliat deleted the extesible-deploy-cmd branch February 16, 2021 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extensibility issues to simplify ability to extend Theia plug-in system issues related to the plug-in system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants