-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Restrict outdent in parens for certain region prefixes #22530
base: main
Are you sure you want to change the base?
Conversation
That breaks sudoku!
which is hard to read and misleading indentation, but is it illegal. The intuition here would be that if you indent the body of your function literal, you intend it to look like a block, and not mashed together with trailing args. Restricting the restriction to
Maybe that is OK, because the colon is special. I noticed that we wanted templates defined with |
dceeef7
to
4362c32
Compare
That breaks scalaz!
where it looks like |
4362c32
to
17d360e
Compare
The spurious timeout. |
b20e6a2
to
dc46f94
Compare
That blocks other corpus tests. Probably it is too opinionated, but I'll try unblocking just to collect the data. |
dc46f94
to
b4a4319
Compare
Possible fix requires comma at end of line if the indented region was introduced by a colon
(and similar prefix tokens).Fixes #22527