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
[Parser]: Labeled block without do diagnostics #29147
Conversation
Continue after skilUntilDeclStmtRBrace
@kitaisreal, going forward, could you add some more information in the commit messages? Right now, the two commit messages in this PR are very similar. There are a couple of options, whichever works for you: (1) make changes as suggested, rebase + force-push (review is still going on) or (2) make changes as suggested in new commits, rebase + force-push after changes are approved, retest and merge. |
Hi, @varungandhi-apple lets make changes in new commits after review, then rebase, force push and retest. |
I was thinking of another strategy for this.
Using this strategy, we can stop emitting |
Fixed code review issues.
Hi, @rintaro totally agree. Fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great overall! Just a few comments.
@swift-ci Please smoke test |
@swift-ci Please smoke test macOS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @kitaisreal !
Improve diagnostics for labeled block without 'do'
More information in ticket https://bugs.swift.org/browse/SR-3867