Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upRename spanned HIR node enums from Foo_ to FooKind #52264
Conversation
rust-highfive
assigned
oli-obk
Jul 11, 2018
rust-highfive
added
the
S-waiting-on-review
label
Jul 11, 2018
This comment has been minimized.
This comment has been minimized.
|
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 |
This comment has been minimized.
This comment has been minimized.
|
As mentioned on the issue, it would make me extremely happy if clippy were also fixed. I'll look through the 1k changed lines tomorrow xD Also you have some tidy failures |
This comment has been minimized.
This comment has been minimized.
|
@oli-obk clean-up for clippy will be addressed as soon as this PR is landed. |
This comment has been minimized.
This comment has been minimized.
|
I was hoping to do a lockstep upgrade. If you do the cleanup, I'll take care of keeping this PR rebased and working and give it a priority so we get it through without breaking clippy at all on master |
csmoe
force-pushed the
csmoe:kind
branch
3 times, most recently
from
9f8d890
to
7c64343
Jul 12, 2018
This comment was marked as resolved.
This comment was marked as resolved.
|
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 |
csmoe
force-pushed the
csmoe:kind
branch
from
7c64343
to
12e2691
Jul 12, 2018
csmoe
referenced this pull request
Jul 12, 2018
Closed
Rename spanned HIR node enums from Foo_ to FooKind #2913
This comment has been minimized.
This comment has been minimized.
|
|
csmoe
and others
added some commits
Jul 11, 2018
oli-obk
force-pushed the
csmoe:kind
branch
from
12e2691
to
c692816
Jul 16, 2018
This comment has been minimized.
This comment has been minimized.
|
I pointed the clippy submodule directly at a branch in the clippy repository in order to make sure this PR doesn't break clippy. Additionally I snuk in a minor test output change (some backticks were added to error messages of rustc) @bors r+ p=1 Superb cleanup and thanks for taking the time to also update clippy! |
This comment has been minimized.
This comment has been minimized.
|
|
bors
added
S-waiting-on-bors
and removed
S-waiting-on-review
labels
Jul 16, 2018
This comment has been minimized.
This comment has been minimized.
bors
added a commit
that referenced
this pull request
Jul 16, 2018
This comment has been minimized.
This comment has been minimized.
|
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 |
This comment has been minimized.
This comment has been minimized.
|
|
csmoe commentedJul 11, 2018
Closes #51968
r? @oli-obk