-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
rustc: rename ty::maps to ty::query. #51550
Conversation
This comment has been minimized.
This comment has been minimized.
We should update rustc-guide: |
@bors r+ |
📌 Commit 5a5c7de has been approved by |
⌛ Testing commit 5a5c7de with merge df21f7aed7f148bc717dea666cc8a7525515d851... |
If we are changing this, why not put it in |
💔 Test failed - status-appveyor |
@Zoxc I plan to, but I'd rather shift around the contexts a bit first. Currently it's all internal to the tcx, and that needs to change, making the typesystem secondary to queries. |
⌛ Testing commit 5a5c7de with merge 5f79de7b8b8f1ec468b50a06fdf8080d16aabfab... |
💔 Test failed - status-appveyor |
@bors retry |
@bors retry |
⌛ Testing commit 5a5c7de with merge d812abff31eb94dd097d1996965850734853e7ea... |
💔 Test failed - status-appveyor |
⌛ Testing commit 5a5c7de with merge 1a891142d0ea837f6486e08ea987de2d20bb8aa0... |
💔 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 |
What's happening here? @bors retry |
rustc: rename ty::maps to ty::query. Should've never been `maps` but "query system/engine" didn't fully settle from the start. r? @michaelwoerister or @nikomatsakis
☀️ Test successful - status-appveyor, status-travis |
Should've never been
maps
but "query system/engine" didn't fully settle from the start.r? @michaelwoerister or @nikomatsakis