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

draw.io extension (GPL-3.0) #51

Closed
hallvard opened this issue Jun 18, 2020 · 10 comments
Closed

draw.io extension (GPL-3.0) #51

hallvard opened this issue Jun 18, 2020 · 10 comments

Comments

@hallvard
Copy link

I would like to integrate draw.io into gitpod, e.g. using this extension: https://github.com/hediet/vscode-drawio

This particular one failed with en error related to registerCustomEditorProvider is not a function. I'm guessing it is using new API for registering custom editors that is not official or not supported by gitpod.

@jankeromnes
Copy link
Collaborator

Hi @hallvard!

I would like to integrate draw.io into gitpod, e.g. using this extension: https://github.com/hediet/vscode-drawio

I wouldn't mind automatically publishing this extension to https://open-vsx.org, except that I don't think GPL-3 extensions are compatible yet, cc @spoenemann (same problem as #49).

This particular one failed with en error related to registerCustomEditorProvider is not a function. I'm guessing it is using new API for registering custom editors that is not official or not supported by gitpod.

Ah, that could indeed be the case. For VS Code API compatibility issues in Gitpod or Theia, please open an issue in https://github.com/eclipse-theia/theia/issues (providing the extension you tried and the error you got is super helpful, thanks).

@jankeromnes
Copy link
Collaborator

@hallvard Good news! The Eclipse board may soon allow all "standard" open-source licensed extensions onto OpenVSX. While no official final decision has been announced yet, we consider this to be very promising, and have decided to accept the draw.io extension now. Added in f4a8b06. Many thanks for your patience!

@hallvard
Copy link
Author

hallvard commented Sep 25, 2020

Nice! Does this mean that the custom editor extension is implemented so this extension actually works? It doesn't seem so from the corresponding issue #6636.

@spoenemann
Copy link
Collaborator

That's a Theia issue. This repository is only about publishing extensions.

@spoenemann
Copy link
Collaborator

@jankeromnes looking at the package.json it seems you need to add a yarn pre-build prepublish command.

Should we also add a section to our README.md about checking prepublish requirements of extensions?

@jankeromnes
Copy link
Collaborator

@jankeromnes looking at the package.json it seems you need to add a yarn pre-build prepublish command.

Aha, thanks for catching that!

Should we also add a section to our README.md about checking prepublish requirements of extensions?

We could. Do you have a general process that you follow to check these? (My process is looking at "scripts" in package.json, and seeing if there is something called "compile" or "build" or "package" that isn't covered by "vscode:prepublish", but maybe that's not the best way.)

@spoenemann
Copy link
Collaborator

looking at "scripts" in package.json, and seeing if there is something called "compile" or "build" or "package" that isn't covered by "vscode:prepublish"

👍

@jankeromnes
Copy link
Collaborator

Publishing worked! 🎉 https://open-vsx.org/extension/hediet/vscode-drawio

But the version string is 999.0.0-alpha.

@spoenemann
Copy link
Collaborator

spoenemann commented Sep 30, 2020

But the version string is 999.0.0-alpha.

Looks like there is a missing build step, but I can't see that from the package.json.

Maybe we should remove the entry and the published extension, otherwise we'd break auto-update for everyone who installs this version.

@hediet
Copy link

hediet commented Jan 17, 2021

Author of the Draw.io extension here. I setup a CI step so that whenever the extension is published to the VS Code marketplace, it is also published to open-vsx!
So no external auto publish should be required anymore.

But the version string is 999.0.0-alpha.

I setup CI to extract the version from the changelog.
The version in package.json is 999.0.0-alpha so that, when I debug my extension, it is never overwritten by any installed extension.

tennox pushed a commit to tennox/publish-extensions that referenced this issue Jan 21, 2022
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

No branches or pull requests

4 participants