Remove io.aviso.writer namespace

[hlship]

Yes, it's an incompatible change, but it simplifies things greatly. I've also updated some documentation to reflect Clojure 1.10.

This was also a chance to update documentation, docstrings, and some code.

[staypufd]

I was just reading about the nREPL project and it’s documentation. If you change this to put only to out then It may not be usuable with the nREPL based on what nREPL says about requirements for Pretty Printers.

https://nrepl.org/nrepl/design/middleware.html down in the section on Pretty Printing it has the requirements.

[hlship]

I'm not sure how these two things relate, or at least, how the changes proposed are in any way a change from the current behavior; essentially, all the Writer code was an alternative to using with-out-str; this PR removes the writer namespace, and makes use of with-out-str.

[staypufd]

Ok - I probably just misunderstand how the nREPL pretty-printer would interface with your awesome pretty-printer.

I think that when I read that nREPL expects a custom printer to return and string and not a value I confused that with how it would interface into yours. But now I see that your returns a string and so it’s fine and having a Writer to write on doesn’t really matter to nREPL.