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

Use correct script while building volar #628

Merged
merged 5 commits into from
Aug 5, 2022

Conversation

wvffle
Copy link
Contributor

@wvffle wvffle commented Aug 1, 2022

  • 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

This PR applies @johnsoncodehk's suggestions from #626.

@filiptronicek
Copy link
Member

Thanks a lot, @wvffle!
Would you be able to look at the build errors with Vue.vscode-typescript-vue-plugin as well? If we fix something about the extension it would be nice to have it build successfully.

@wvffle
Copy link
Contributor Author

wvffle commented Aug 2, 2022

@filiptronicek, I've tried to look at the errors. It is... weird... If I install the dependencies in the root directory with npm, there are typescript errors but when I install them with pnpm - it builds fine. But the Vue.vscode-typescript-vue-plugin plugin needs its own dependencies to be installed. If installed with pnpm - an ELSPROBLEMS error is thrown but when installed with npm, it works fine.

What I did feels very hacky but it seems to work and package the extension:
image

@johnsoncodehk
Copy link

@wvffle Vue.vscode-typescript-vue-plugin actually cannot pack via script, this is a limitation of vsce, I mush to do these steps manually:

  1. run pnpm install && npm run build in repo root
  2. remove "dependencies": { "typescript-vue-plugin-forward": "file:typescript-vue-plugin-forward" }, in extensions/vscode-typescript-vue-plugin/package.json.
  3. run npm run pack in extensions/vscode-typescript-vue-plugin.
  4. add back "dependencies": { "typescript-vue-plugin-forward": "file:typescript-vue-plugin-forward" },

extensions.json Outdated Show resolved Hide resolved
@wvffle
Copy link
Contributor Author

wvffle commented Aug 2, 2022

@johnsoncodehk thanks for explaining the build steps!

@filiptronicek filiptronicek added the bug Something isn't working label Aug 5, 2022
@filiptronicek
Copy link
Member

Looking better now, @wvffle!

Could you take a look at the /bin/sh: 1: Syntax error: redirection unexpected error? [1]

@wvffle
Copy link
Contributor Author

wvffle commented Aug 5, 2022

Could you take a look at the /bin/sh: 1: Syntax error: redirection unexpected error? [1]

Should work now

Copy link
Member

@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.

Many thanks, @wvffle!

LGTM ✅

@filiptronicek filiptronicek merged commit c46e989 into EclipseFdn:master Aug 5, 2022
@wvffle
Copy link
Contributor Author

wvffle commented Aug 5, 2022

@filiptronicek, there seems to be an error while publishing volar to the repository: https://github.com/open-vsx/publish-extensions/runs/7693396759?check_suite_focus=true#step:9:6797, could you take a while and look at it? :)

@filiptronicek
Copy link
Member

Thanks for noting this, @wvffle, I have taken a look at this and sadly it seems it's going to be blocked until open-vsx.org deploys next, which I'm not sure when it will be :/.

@anbraten
Copy link
Contributor

@filiptronicek I think the Vue namespace is currently owned by @lukashass if that helps: vuejs/language-tools#1255
Could we help you somehow to unclaim it?

@filiptronicek
Copy link
Member

@anbraten if I'm not mistaken, you cannot just get ownership of a namespace by creating it. You only are assigned as a member, you cannot add others to it therefore not an owner - to be an owner you need to claim the namespace, first. @open-vsx is kind of the BDFL of https://open-vsx.org, it can publish any extension at any time under any namespace. It shouldn't be the problem here. I will try to discuss with the team at Eclipse if they have any logs about the publishing errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants