Nemo157

Hmmm, I don't see how to get it down to 1 line, my expectation was 3 ```rust fn value_to_string(value: &T) -> Result { let mut output = String::new(); value.serialize(toml::ser::ValueSerializer::new(&mut s))?;...

(even with the config `allow-private-module-inception` enabled the warning occurs, and IMO that config should be on by default).

The item is _not_ available at `foo::foo::Foo` though, that is a private module, it makes no impact on the public API. That part of the lint description seems to be...

Thinking about this a little more, the intent of this lint afaict is to catch beginners coming from languages like C# or C++ where you re-declare the current module in...

I would also prefer if reordering of trait impls follows the order of the trait definition (at least for locally defined traits). I have a case where there are multiple...

Looks like it wasn't seen for some reason, production logs from the registry checks before/after the publish event: ``` 2022-11-17T17:25:29.611096Z DEBUG docs_rs::utils::daemon: Checking new crates 2022-11-17T17:25:30.357202Z DEBUG docs_rs::utils::daemon: 0 crates...

> when fixed, is it worth running our consistency-check command? Does it still work? Not sure, this should be very rare, I think it only happens when a crate is...

You could have an intermediate wrapper that doesn't forward shutdown to the underlying file, just flushes it and reports that it has been shutdown. ```rust GzipEncoder ```

That's something I'd thought of doing for a while but never got round to. I was also thinking it'd be good to extract the generic state machine IO impls into...

Out of the docs I have, the largest file is ``` 91513860 docs/web-sys/0.3.35/x86_64-pc-windows-msvc/src/web_sys/opt/rustwide/target/x86_64-pc-windows-msvc/debug/build/web-sys-caeee69dd050a8ab/out/bindings.rs.html ``` there's a few of those for the different platforms, then it drops down to a handful...