-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
gh-118090: Improve error message for empty type param brackets #118091
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
Conversation
Ubuntu failures are unrelated: connection timeout. |
there's also a failure about generated files not being up to date, though, which looks real ;) |
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.
Thanks! Left a suggestion for improving the error.
Thanks for the review! I will also wait for @pablogsal or @lysnikolaou review as code owners :) |
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.
LGTM! Makes sense we had to remove the forced token stuff from the first pass.
class A[]: ...
syntax error message #118090