-
Notifications
You must be signed in to change notification settings - Fork 18
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
Formatting errors aren't properly handled when auto_format
is enabled
#32
Comments
Any updates? |
Bump, this is critical for me, anyone could reproduce it? |
Really |
@vladfaust Sorry for the late response, I'm organizing the issues and PRs to track them easily 😅 Check https://github.com/orgs/crystal-lang-tools/projects/1?fullscreen=true 😉 |
@fgimian I would be happy if you could help me with that 😢 |
Yep, I can reproduce it as follows. Ensure that
I'm sure we can fix this quite easily. I personally don't like If I get a little spare time, I'll try to assist with a fix for this extension. Cheers |
The problem is simply that there's no exception handling in https://github.com/crystal-lang-tools/sublime-crystal/blob/master/crystal_format.py if things go wrong. Cheers |
auto_format
is enabled
I'll be sending a pull request that fixes this in a sec guys 😄 |
Corrected handling of invalid source code when auto formatting (fixes #32)
Given this file:
This is what I've got in the console:
Basic formatting works, but if there is an error in the code, it's not handled properly - no message is given except these errors in the console.
The text was updated successfully, but these errors were encountered: