It's not allowed to have a statement as the last def. In particular, statements with the await bang shouldn't be considered as empty return because they always desugar into expressions. So I removed EmptyDefsFinal which led to noticeable parsing simplification.
[x] split changes into meaningful commits for easier reviewing
[x] add a new test for the specific case
[x] thoroughly check all snapshots. they all are failed, but it looks like mostly due to changes in regions (probably there's an issue with the regions)
[ ] use indoc! in suffixed tests going to add in a separate pr
kdziamura
commented
3 days ago
closes #6656
It's not allowed to have a statement as the last def. In particular, statements with the await bang shouldn't be considered as empty return because they always desugar into expressions. So I removed
EmptyDefsFinalwhich led to noticeable parsing simplification.Zulip thread
usegoing to add in a separate prindoc!in suffixed tests