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

feat(plugins): Use meta.yaml content generated by script #717

Closed
wants to merge 1 commit into from

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Nov 27, 2020

What does this PR do?

  • Use icons specified inside the vsix files
  • Also update field information based on data from the linked vsix

Screenshot/screencast of this PR

What issues does this PR fix or reference?

eclipse-che/che#17029

How to test this PR?

build the registry and browse plug-in registy

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Change-Id: I6ee177116b327d479ad3ea8808ebbfb3251300e6
Signed-off-by: Florent Benoit fbenoit@redhat.com

Copy link
Contributor

@ericwill ericwill left a comment

Choose a reason for hiding this comment

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

Are we in the clear from a licensing perspective?

@benoitf
Copy link
Contributor Author

benoitf commented Nov 27, 2020

I would say yes as these icons are provided to be exposed by a registry/marketplace/etc.
These icons are already provided by open-vsx as well

@ericwill
Copy link
Contributor

I would say yes as these icons are provided to be exposed by a registry/marketplace/etc.
These icons are already provided by open-vsx as well

OK, I agree

@benoitf benoitf marked this pull request as draft December 2, 2020 10:34
@benoitf benoitf force-pushed the fix-values branch 2 times, most recently from 4d8ac43 to a762b3f Compare December 2, 2020 12:25
also extract icons

Change-Id: Ibceebba2bf03877d759886e2f4d21d587a45a18b
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@vitaliy-guliy
Copy link
Contributor

vitaliy-guliy commented Dec 2, 2020

Why is it better to have the icons inside our repository?
I'm just wonder how it's related to generating of meta.yamls

It there a script used to fetch those icons from .vsix files?

In Che we use docker image with plugin registry. The build script could fetch all those icons from .vsix and place them named {publisher}.{id}.png to /images directory inside docker image. And yes, all the meta.yamls for vs code extensions should be preprocessed with changing the icon path.
This gives us not to store the icons in this repository but have them inside the container.
WDYT?

@benoitf
Copy link
Contributor Author

benoitf commented Dec 2, 2020

@vitaliy-guliy: all that you're saying is correct
here it's like the last intermediate step before removing all meta.yaml and images and make it only available in the final image (build time fetch)

The script is there #721 (it extracts vsix/icons automatically and create meta.yaml)

@benoitf benoitf changed the title feat(plugins): Use icons specified inside the vsix files feat(plugins): Use meta.yaml content generated by script Dec 2, 2020
@benoitf
Copy link
Contributor Author

benoitf commented Dec 4, 2020

closing as it was to check diff between what is generated and what is available

PR to follow is #721

@benoitf benoitf closed this Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants