Clarify the base16 aspect vs bytes aspect of WKB

[josevalim]

No description provided.

[josevalim]

Hi @bryanjos! Thanks for the great project! :)

I was a bit confused that WKB did not return a binary and we were base16 encoding it by default. One option is to deprecate encode! and some of the branches in decode, but assuming it is probably too late for that, I submitted a PR that changes the docs to better clarify where and how it handles both formats.

[wojtekmach]

Should we deprecate WKB.encode, WKB.decode("00" <> ...), and WKB.decode("01" <> ...) with a message pointing towards functions that work on bytes? Happy to send a PR.

[bryanjos]

Hey @josevalim thanks for the PR! I'm definitely open to deprecating what needs to be in order to make things better and less confusing. Same thing with what @wojtekmach mentioned. Sorry it took me so long to get to this. But I'm happy to accept PRs for anything that can help in that effort.