-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
HirIdification: almost there #62091
HirIdification: almost there #62091
Conversation
Not the worst problem to have =P @bors delegate+ |
✌️ @ljedrz can now approve this pull request |
@bors r=Zoxc |
📌 Commit 87438a1 has been approved by |
⌛ Testing commit 87438a1 with merge dbfeb9da46149eb475c6e5bc0c27b12d3a1b661a... |
💔 Test failed - checks-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 |
@bors retry |
… r=Zoxc HirIdification: almost there I'm beginning to run out of stuff to HirIdify 😉. This time I targeted mainly `hir::map::{find, get_parent_node}`, but a few other bits got changed too. r? @Zoxc
Rollup of 7 pull requests Successful merges: - #61814 (Fix an ICE with uninhabited consts) - #61987 (rustc: produce AST instead of HIR from `hir::lowering::Resolver` methods.) - #62055 (Fix error counting) - #62078 (Remove built-in derive macros `Send` and `Sync`) - #62085 (Add test for issue-38591) - #62091 (HirIdification: almost there) - #62096 (Implement From<Local> for Place and PlaceBase) Failed merges: r? @ghost
I'm beginning to run out of stuff to HirIdify 😉.
This time I targeted mainly
hir::map::{find, get_parent_node}
, but a few other bits got changed too.r? @Zoxc