-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Add dedicated testcase for #15881 #21376
Add dedicated testcase for #15881 #21376
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
r? @cmr |
// option. This file may not be copied, modified, or distributed | ||
// except according to those terms. | ||
// | ||
// ignore-lexer-test FIXME |
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 ignore should no longer be here
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.
Yup, I've removed the line (and updated the copyright notice date)
Also enable an ignored test case, closes rust-lang#15881
a4cab28
to
b8efaa6
Compare
Very well. |
Isn't @cmr away because of studies? |
I am not sure if this issue is still relavant after the new model parser got merged, as this issue involves the antlr lexer, which is obselete now, I guess? |
⌛ Testing commit b8efaa6 with merge 5870f25... |
@bors: retry |
⌛ Testing commit b8efaa6 with merge 3b9da16... |
💔 Test failed - auto-mac-64-nopt-t |
@bors: retry |
⌛ Testing commit b8efaa6 with merge 95c237a... |
💔 Test failed - auto-win-32-opt |
@bors: retry |
⌛ Testing commit b8efaa6 with merge b8c112f... |
This PR adds a dedicated regression test for #15881 as @cmr suggested.