llogiq
llogiq
Sorry it took so long to review, but this seems good to merge. Let's hope we're in time. @bors r+
Hey @Jarcho, do you plan on getting this merge-ready any time soon?
Sorry for having this wait for so long, but it looks good to me, so ping me once the conflicts are resolved.
Sorry for another rebase needed.
Sorry this is taking so long, I am unfortunately quite busy at the moment. I'll ping you once I've run the check so you don't need to rebase more than...
I finally came around to do a lintcheck run, and it looked OK. So r=me after a rebase and the docs fix @bitfield suggested.
No problem. @bors r+
@nyurik: Currently the lint only catches HashMaps, but I'd welcome a PR to change that along with the name.
I think you need to re-bless the tests.
@rustbot author