Skip to content

Automatic Semicolon Insertion issue #5

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

Closed
Luxcium opened this issue Sep 19, 2019 · 3 comments
Closed

Automatic Semicolon Insertion issue #5

Luxcium opened this issue Sep 19, 2019 · 3 comments

Comments

@Luxcium
Copy link

Luxcium commented Sep 19, 2019

I was editing my Issue and my corrector crashed my browser...

Ok so I will try to be short and concise: the problem is on line 4 to 5 (11 & 12 on the image) and the related code I was working on is on GitHub commit: aeefa22e38

Capture d’écran, le 2019-09-19 à 04 32 14

I am using a clean and fresh out of the box version of VSCode "portable" with only «ms-vscode.vscode-typescript-next» installed

In the code in my example, the missing semicolon would have not really been an issue because I am using the property combinedBalancesbut the same problem happens if the property reference a function (i.e. using a method) my code will require to use const/let/var thus avoiding this issue altogether (or I could have added the semicolon myself) ut the problem is that I rely on autocompletion/prettier/linter to do that part of the job...

I fell like its a bug and I wish you will have enough pieces of information to find a solution maybe I should add the semi or maybe my linter should do it but I feel the autocompletion should have not added a problem right away...

Tell me how I can be part of the solution in a constructive way so you can fix the problem...

LANGUAGE: The best evolution of ES262 specification ever (TypeScript)
IDE: The amazing VISUAL STUDIO CODE
Version: 1.38.1
Commit: b37e54c98e1a74ba89e03073e5a3761284e3ffb0
Date: 2019-09-11T13:31:32.854Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0

@mjbvz
Copy link
Collaborator

mjbvz commented Sep 19, 2019

I don't think this is an issue with this extension. Please file an issue against typescript itself

@mjbvz mjbvz closed this as completed Sep 19, 2019
@Luxcium
Copy link
Author

Luxcium commented Sep 20, 2019

What type of problem is it. ? I don’t know how to describe it more than I did here... can you point me in a direction before I take the time to open a ticket (issue) with them ...

@mjbvz
Copy link
Collaborator

mjbvz commented Sep 20, 2019

Try filling out the TypeScript issue template. Describe what you were doing, what the expected behavior was, and what went wrong

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

2 participants