Update with latest upstream code

[stereokai]

Hi @Ericbla, I forked your binary-parser-encoder fork and updated it by cherry picking all the commits from the original project that were missing here. I also got all current tests passing both for the parser and the encoder.

I'll be happy to commit for a little while to ensure your encoder project gets merged into binary-parser. As the original developer, your input and help in understanding your original motives and challenges would make a huge difference, so please let me know if you'd like to collaborate on this again :)

CC @keichi

[Kreijstal]

@stereokai Thank you so much, thank you!

[kristian]

@stereokai hope it is fine that I published a unmodified fork of your fork (I guess we are talking about a sub-sub-tine aka a „splinter“ at this point in time*) to NPM, I simply named it „binary-parser-encoder-bump“:

https://github.com/kristian/binary-parser

I wanted to base another one of my projects (that I am also planning on publishing) based on the parser / encoder version 2.2.1, so I needed the version on NPM. Please feel free to reach out, in case of any concerns! Thanks for your efforts. @Kreijstal, @keichi FYI.

* if the original fork of binary-parser to binary-parser-encoder was the fork, the fork of @Kreijstal was the tine, @stereokai yours was the sub-tine and mine is the sub-sub-tine, or a spliter of the original fork ;)

[stereokai]

@kristian my pleasure.

This is hilarious. You could also say mine is the fork in the tine and yours is the tine on the fork in the tine 😁

Wonder if @Ericbla or @keichi will ever take a look at it.

[kristian]

@stereokai

This is hilarious. You could also say mine is the fork in the tine and yours is the tine on the fork in the tine 😁

haha, nice! But joking aside, writing this summary:

This is a fork of Tom Alon's updated fork of Eric Blanchard's binary-parser-encoder fork adding encoding capabilities to the original binary-parser by Keichi Takahashi. Until the the encoding feature is merged in baseline of original project, this fork / branch is published under the name: binary-parser-encoder-bump on NPM.

… literally took more time than releasing the package to NPM, lol! In hopes of the original encoding feature gets merged into the upstream repository at some time, so the fork-ception can finally come to an end! ;) Cheers mate!

[Kreijstal]

Finally... an official fork

[Kreijstal]

@stereokai your fork doesn't have issues enabled

[stereokai]

@Kreijstal opened