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

@pulumi/azure-native crashes TypeScript compiler #2251

Closed
michaellperry opened this issue Feb 18, 2023 · 2 comments
Closed

@pulumi/azure-native crashes TypeScript compiler #2251

michaellperry opened this issue Feb 18, 2023 · 2 comments
Labels
customer/feedback Feedback from customers resolution/by-design This issue won't be fixed because the functionality is working as designed

Comments

@michaellperry
Copy link

I love this service! It's a joy to work on infrastructure using the languages I love.

I ran into a problem with a TypeScript project on Azure. If @pulumi/azure-native is installed, tsc crashes. This prevents the language support from loading in VS Code. As a result, I don't have TypeScript intellisense.

This is documented in this GitHub issue.

I'm going to try moving my project to @pulumi/azure as a workaround.

@michaellperry michaellperry added customer/feedback Feedback from customers needs-triage Needs attention from the triage team labels Feb 18, 2023
@aureq aureq transferred this issue from pulumi/pulumi-cloud-requests Feb 18, 2023
@aureq
Copy link
Member

aureq commented Feb 18, 2023

@michaellperry Glad to here you're enjoying Pulumi.

On the crash with VS Code, I had a similar issue a few weeks ago and I was able to resolve it by increasing the maximum memory for the TS Server.

In the VS Code preferences: TypeScript > Tsserver: Max TS Server Memory and set the value to 4096 (this one worked for me) or maybe higher. Then close VS Code and reopen your project (a reboot of your env may be necessary).

Let us know please.

@kpitzen kpitzen added resolution/by-design This issue won't be fixed because the functionality is working as designed and removed needs-triage Needs attention from the triage team labels Feb 22, 2023
@kpitzen
Copy link
Contributor

kpitzen commented Feb 22, 2023

Hi @michaellperry ! This is on our radar as one of the major improvements coming in Azure Native v2 - I'm going to mark this as "by design" for now, since the v1 spec includes every version of every resource for every API - that manifests itself in the form of poor performance (to the point that it can crash the language server in some cases).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer/feedback Feedback from customers resolution/by-design This issue won't be fixed because the functionality is working as designed
Projects
None yet
Development

No branches or pull requests

3 participants