-
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
Got rid of repetitions (such as rustc_ast::ast::) in librustc_ast #71199
Conversation
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 |
r? @petrochenkov (who wasn't that keen on this, as per #70621 (comment) - I don't want to just override that) (but also, CI doesn't pass) |
Yes i'm aware, investigating the failure (which seems odd) |
Yeah, I just don't like a bunch of names existing in the crate root - some from the AST proper, some are just modules from the I don't think we have other crates organized like this? Usually crate root modules are not very populated. |
Sigh, we already have |
Ye as I was doing these changes, I saw a similar pattern with hir |
r? @eddyb who wanted to do this, I won't stand in the way. |
@bors try |
⌛ Trying commit 17a5e3e8f514a947945ded2c7652d670401f4238 with merge b5d8c557b2ae4fb6157c4b0d05d15e8ca0fe39ca... |
💔 Test failed - checks-azure |
☔ The latest upstream changes (presumably #70820) made this pull request unmergeable. Please resolve the merge conflicts. |
7ed48d2
to
1e1a163
Compare
Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem. 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 |
Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem. 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 |
9eee4ee
to
102756b
Compare
☔ The latest upstream changes (presumably #71292) made this pull request unmergeable. Please resolve the merge conflicts. |
There's a merge conflict now, needs a rebase. |
Gonna close it as it's not important and rework it some other time. |
rust_ast::ast => rustc_ast Rework of rust-lang#71199 which is a rework rust-lang#70621 Still working on this but just made the PR to track progress r? @Dylan-DPC
rust_ast::ast => rustc_ast Rework of rust-lang#71199 which is a rework rust-lang#70621 Still working on this but just made the PR to track progress r? @Dylan-DPC
Rework of #70621
r? @eddyb
credit to @Lezzz for the original pr :)