-
-
Notifications
You must be signed in to change notification settings - Fork 187
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
Deploys are broken #174
Comments
Deploys are working again for npm, but for the VSCode extension there is a bug I was not able to track down yet: The node_modules folder is missing in the build of the extension, so it will never start. You can revert to 99.0.33, that one should work. |
how can we help? is it a problem with orta's action or with us? |
I guess it's the actions. Since the deployment is tightly integrated with Orta's monorepo-deploy-repo, we cannot do much there right now I think. |
Hrm, it should be running npm install before each deploy - but it doesn't sound like it |
is it possible that the dependencies for svelte-vscode are wrong? This list seems very short:
|
Scanned the packages, the following are imported packages: Even if not - it would at least still have to create a node_modules folder with those packages in it. And second, if that was the problem, why did it not fail before? |
It is definitely running |
Yeah it's really strange. Could it go missing during the deploy step with |
Last deploy run now shows some npm errors about extraneous packages. All of those are from the If that is true, we would need to do two things:
|
...
"dependencies": {
"svelte-language-server": "file:../language-server", // <--- file:
... |
Got it deploying properly again.
It's all still in a rough state which needs some cleanup, but the main problem is fixed. I ended up not adding a bundle/minify step, so that's still to do, but separately. |
👍 thanks @dummdidumm! |
I shipped a commit first without the prune, and then another which drops the |
Working regularly again. |
My bad, still figuring it out, better to have an issue I can "close" when done
The text was updated successfully, but these errors were encountered: