Rain

Results 514 comments of Rain

(yeah that's what I meant, just following the contract and documenting that it is followed)

It does, yes! So it looks like the only thing to do is to verify that the impla follow the TrustedLen contract, update them if they don't, and document that...

Not sure why it didn't get updated, but the Travis build is complete.

I wanted `enable_ansi_support` without the rest of `ansi_term`, so I've included the no-op function and published it as [a separate crate](https://crates.io/crates/enable-ansi-support) ([repository](https://github.com/sunshowers/enable-ansi-support)).

Yes, I think including those two traits is reasonable since they have defined semantics for undirected graphs.

Thank you! > Good point about the self loops. I'll filter `self` from the `Outgoing` iterator unless there's a better way you can think of? That seems totally reasonable. If...

The CI error looks like it's coming from a dependency bumping its MSRV.

Thanks for the issue! I'm totally open to this as an optional feature, since almost all paths are valid UTF-8. For the PR, could you include some benchmarks + results...

> It's not ready for production so it definitely should be optional. Could you elaborate on what this means? This is a bit concerning to me.

Thanks! It also looks like it switches to std for fewer than 64 bytes. I'm really curious how much of a benefit this is in real world use because most...