Should with_mocked_bindings() follow withr argument style?

[jameslairdsmith]

I got momentarily confused using with_mocked_bindings() because I'm used to using withr, where the with_* functions have the temporary state as the first argument, and the code= as the second argument. withr also usually makes new= its own argument rather than using the dots. I don't know if it's worth harmonising with_mocked_bindings() with that pattern?

[hadley]

I think it's probably too late for that 😞