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

Add swyddfa.esbonio (#741) #742

Closed

Conversation

tchernobog
Copy link
Contributor

@tchernobog tchernobog commented Feb 12, 2024

  • I have read the note above about PRs contributing or fixing extensions
  • I have tried reaching out to the extension maintainers about publishing this extension to Open VSX (if not, please create an issue in the extension's repo using this template).
  • This extension has an OSI-approved OSS license (we don't accept proprietary extensions in this repository)

Description

Add swyddfa.esbonio (#741).

Author has manifested interest in publishing to OpenVSX once this hits version 1.0, but I would like a temporary solution for now as 1.0 might be several months away.

Copy link
Collaborator

@filiptronicek filiptronicek left a comment

Choose a reason for hiding this comment

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

Hi @tchernobog 👋, thanks for the contribution.

I am happy to add the extension, but the size we build is about 180KB, while the official extension is somewhere about 1.4MB. I think there are some build steps we're missing

@tchernobog tchernobog force-pushed the extension/swyddfa.esbonio branch 6 times, most recently from cda168d to 6fb78c2 Compare February 13, 2024 21:50
@tchernobog
Copy link
Contributor Author

tchernobog commented Feb 13, 2024

Thanks @filiptronicek for catching this. I fixed the build, but I still have one problem.

As far as I can tell, the code in open-vsx/publish-extensions tries to infer the latest version of an extension using the github repository tags.

Unfortunately, the extension uses a non-standard format. The latest tag is currently for an older release from a different branch than mainline (called here develop).

Here we need to consider only those tags that match esbonio-vscode-extension-v*. The latest version is 0.92.1.

The latest tag is for esbonio-language-server-v0.16.4 which in turn contains the code for a very old 0.11.0 extension.

During the build it's easy for me to fix the checked out tag, but as the context is already set at that point, the wrong version is published:

{
  "context": {
    "msVersion": "0.11.0",
    "msLastUpdated": "2023-03-06T22:51:28.410Z",
    "msInstalls": 13902,
    "msPublisher": "swyddfa",
    "version": "0.11.0",
    "repo": "/tmp/repository/main",
    "ref": "esbonio-language-server-v0.16.4"
  }
}

How can this be fixed? I don't see anything in the JSON schema.

@filiptronicek
Copy link
Collaborator

Thanks for looking into this further, @tchernobog. Does this mean that the currently built package for esbonio-0.92.1.vsix is incorrect, or did your git checkout tricks work out in the custom instructions?

The setup here looks quite complex and difficult to maintain, so I think it would be best to publish the current latest version manually and wait for the author to publish the 1.0 release themselves as you mentioned.

@tchernobog
Copy link
Contributor Author

tchernobog commented Feb 17, 2024

@filiptronicek the results in term of package seem fine.

The only problem is that the deduced version seems wrong (0.11.0). If the result of this run is indeed esbonio-0.92.1.vsix then this is working well, if it deduces it as esbonio-0.11.0.vsix then it's wrong.

The build job seems fine: https://github.com/open-vsx/publish-extensions/actions/runs/7893644227/job/21542661844?pr=742#step:9:418

The question is if the context is still used when publishing somehow, because it shows a different version: https://github.com/open-vsx/publish-extensions/actions/runs/7893644227/job/21542661844?pr=742#step:9:142

If the context is ignored while pushing to the marketplace, then this change should be fine.

@filiptronicek
Copy link
Collaborator

I see the issue now, thanks for pointing it out.

When publishing, this context is not used and the only version info, which is what is actually listed in the packaged extension (Open VSX takes care of that).

I am publishing v0.92.1 now

@filiptronicek
Copy link
Collaborator

Published: https://open-vsx.org/extension/swyddfa/esbonio 🎉

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.

2 participants