rm safe-buffer

[jimmywarting]

part of brave/brave-browser#5490

(Always require latest Buffer module)

[themasch]

Thanks a lot for the pull requests, getting rid of a dependency sounds great!

I am out of the loop with node, what nodejs version do we require for safe-buffer to be not helpful anymore? Do we need a new major release with a new minimum node version requirement to support this?
Also, seems like travis is finally defunct for this repo, I guess someone need to move that to github actions...

[jimmywarting]

what nodejs version do we require for safe-buffer to be not helpful anymore?

I asked this question also a while back... feross/safe-buffer#24 when you only support v6+ then you don't need it

I guess someone need to move that to github actions...

👍

[jimmywarting]

if you like perhaps switch to a ESM only package.

[themasch]

I just merged that adds github actions for running test. Could you please merge those changes in this branch. I ran the test locally with node12, and do not expect any breakage, but just to be safe...here

I asked this question also a while back... feross/safe-buffer#24 when you only support v6+ then you don't need it

okay, thats a long time ago...

[jimmywarting]

Could you please merge those changes in this branch

I did, didn't happen so much new stuff...