-
Notifications
You must be signed in to change notification settings - Fork 13k
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 inline attributes to syntax_pos #50294
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
Could the |
@bors r=alexcrichton |
📌 Commit 2bc67a5 has been approved by |
⌛ Testing commit 2bc67a50de19eb4b5eed7c78f34aa965e87c474a with merge 12f841893f0e2f796b511d0cda5926d39913d056... |
💔 Test failed - status-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
1 similar comment
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@Zoxc Have you checked if the linker error in #50294 (comment) is fixed? The current diff seems to be exactly the same as the diff of 12f841893f0e2f796b511d0cda5926d39913d056. |
I haven't changed anything (or reproduced the linker error). I've just rebased the branch. |
OK let me change it back to |
I've removed the changes to This just leaves the |
@bors: r+ |
📌 Commit 7e53f41 has been approved by |
⌛ Testing commit 7e53f41 with merge 601f2eca1fc2ce6273cd6b346df45928b059dc2f... |
💔 Test failed - status-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
1 similar comment
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The existing
impl PartialOrd<InternedString> for InternedString
causes an error in some of my branches for some reason...