-
-
Notifications
You must be signed in to change notification settings - Fork 540
docs(README.md): add troubleshooting tip for syntax errors #1201
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
Conversation
as discussed in TypeStrong#903. I had to search for awhile to find that issue and figure out what was going on, and there were some duplicate issues, so I think it would help people to explain this directly in the README.
Nice, thanks for writing this up. We already have this section explaining Should we also explain how to identify the difference between a TypeScript syntax error and a node syntax error? They have different names, so once you know where to look, it's easy and reliable to tell them apart. |
I don't think And yeah I think it would be good to merge in what I wrote because I'm not sure the existing explanation there would have been enough to make the light go off in my head. "... cannot be fixed by TypeScript or ts-node. These are runtime issues with your code" sounds like it's saying the mistake is in your code itself, not the transpilation settings. |
If nothing else adding |
Okay I merged my explanation into the syntax error section, see what you think |
Thanks, I haven't forgotten about this. I'm also working on creating a documentation website in #1207 I may roll these changes into a dedicated website page about errors and how to interpret them. Not sure yet; it's all a work-in-progress. |
I ended up adding this to the new "Troubleshooting" page: https://typestrong.org/ts-node/docs/troubleshooting#unsupported-javascript-syntax I'm going to close this to keep the issue tracker tidy, but please feel free to suggest edits or improvements. It's been a while since I've looked at this so I may have neglected something obvious, even if we already discussed it. Thanks again for sending this PR. |
as discussed in #903. I had to search for awhile to find that issue and figure out what was going on, and there were some duplicate issues, so I think it would help people to explain this directly in the README.