-
Notifications
You must be signed in to change notification settings - Fork 224
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
[Scala3] Produce indentation errors #2427
Comments
I think this is actually fine, this way things like Scalafmt can parse it and fix to be of correct indentation. Unless this causes any issues, I would leave as is. It's better to be more permissive than the compiler here. |
Actually, disregard my comment, |
Actually, this is ok for scalafmt, as it will adjust the code and show the user what is the issue. We might want to have ways to expose warnings, which don't fail the compilation in this case. |
@kitbellew I would keep it open if it wasn't fixed. At least it might be useful in rare cases for Metals |
so, to clarify, you'd like it be fixed, not just kept open, right? 😉 |
Yep, ok. I will look at it |
Currently, parser doesn't check if indentation is correct.
The following is parsed without errors as
Source(Defn.Object(Foo), Defn.Def("bar"))
while compiler doesn't accept it:The text was updated successfully, but these errors were encountered: