-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Fixed #48425 : Various functions taking a TyCtxt
and a Span
should be taking a TyCtxtAt
#48651
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
Fixing conflicts. |
TyCtxt
and a Span
should be taking a TyCtxtAt
r? @oli-obk @nikomatsakis can you add me to the list of ppl who can give bors commands? I can't do that yet, even though I can be assigned as a reviewer |
@PramodBisht please squash the commits into one. You can do that with r=me with commits squashed |
@bors delegate=oli-obk |
✌️ @oli-obk can now approve this pull request |
2 similar comments
✌️ @oli-obk can now approve this pull request |
✌️ @oli-obk can now approve this pull request |
@oli-obk I am not able to |
Huh? Merge commits shouldn't matter. Can you paste the entire error? |
@oli-obk can you come on IRC, can we discuss there? |
@oli-obk right after running
master is in sync with main |
Sorry I'm on vacation. This is perfectly normal. Just look through the changes, remove all the things you did not change, |
@oli-obk hmm. you ought to have them soon. =) |
c9d2986
to
6fdf637
Compare
@oli-obk All set, could you please check again :) |
Jup looks good now @bors r+ |
📌 Commit 6fdf637 has been approved by |
Fixed rust-lang#48425 : Various functions taking a `TyCtxt` and a `Span` should be taking a `TyCtxtAt` Hi @oli-obk I have done some code refactoring to fix rust-lang#48425, Please let me know if anything else is required on this.
Fixed rust-lang#48425 : Various functions taking a `TyCtxt` and a `Span` should be taking a `TyCtxtAt` Hi @oli-obk I have done some code refactoring to fix rust-lang#48425, Please let me know if anything else is required on this.
Fixed rust-lang#48425 : Various functions taking a `TyCtxt` and a `Span` should be taking a `TyCtxtAt` Hi @oli-obk I have done some code refactoring to fix rust-lang#48425, Please let me know if anything else is required on this.
Hi @oli-obk
I have done some code refactoring to fix #48425, Please let me know if anything else is required on this.