Repository navigation
Conversation
|
@bors try |
This comment has been minimized.
This comment has been minimized.
[crater] make SEMICOLON_IN_EXPRESSIONS_FROM_MACROS a hard error
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
@craterbot check |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🎉 Experiment
Footnotes
|
|
If my math is right that's 4% of all crates, which IMO is way too huge to even consider. We also don't really want to break |
|
Closing the PR as we got the data, discussion is happening in #162872. |
make semicolon_in_expressions_from_non_local_macros not report-in-deps See #162872 for context. This new lint was made immediately report-in-deps without a crater run. I guess people assumed that the ecosystem fallout would be small, but no data was gathered to confirm that hypothesis. We now have data showing that the ecosystem fallout is [gigantic](#162768 (comment)), so let's remove the report-in-deps from this lint. This should be backported to the 1.100 beta. (1.99 already has this patch.) Cc @joshtriplett
…ps, r=mati865 make semicolon_in_expressions_from_non_local_macros not report-in-deps See rust-lang#162872 for context. This new lint was made immediately report-in-deps without a crater run. I guess people assumed that the ecosystem fallout would be small, but no data was gathered to confirm that hypothesis. We now have data showing that the ecosystem fallout is [gigantic](rust-lang#162768 (comment)), so let's remove the report-in-deps from this lint. This should be backported to the 1.100 beta. (1.99 already has this patch.) Cc @joshtriplett
…ps, r=mati865 make semicolon_in_expressions_from_non_local_macros not report-in-deps See rust-lang#162872 for context. This new lint was made immediately report-in-deps without a crater run. I guess people assumed that the ecosystem fallout would be small, but no data was gathered to confirm that hypothesis. We now have data showing that the ecosystem fallout is [gigantic](rust-lang#162768 (comment)), so let's remove the report-in-deps from this lint. This should be backported to the 1.100 beta. (1.99 already has this patch.) Cc @joshtriplett
make semicolon_in_expressions_from_non_local_macros not report-in-deps See #162872 for context. This new lint was made immediately report-in-deps without a crater run. I guess people assumed that the ecosystem fallout would be small, but no data was gathered to confirm that hypothesis. We now have data showing that the ecosystem fallout is [gigantic](#162768 (comment)), so let's remove the report-in-deps from this lint. This should be backported to the 1.100 beta. (1.99 already has this patch.) Cc @joshtriplett
Rollup merge of #163364 - RalfJung:semicolon-no-report-in-deps, r=mati865 make semicolon_in_expressions_from_non_local_macros not report-in-deps See #162872 for context. This new lint was made immediately report-in-deps without a crater run. I guess people assumed that the ecosystem fallout would be small, but no data was gathered to confirm that hypothesis. We now have data showing that the ecosystem fallout is [gigantic](#162768 (comment)), so let's remove the report-in-deps from this lint. This should be backported to the 1.100 beta. (1.99 already has this patch.) Cc @joshtriplett
Crater experiment, see Zulip for context.