Fix bug with 2 separate `PDFJS` objects, add more Node/Bun tests in CI

Open Electroid opened this issue 9 months ago • 1 comments

This PR fixes a bug where there were 2 separate PDFJS objects, and adds support to run tests in Bun.

https://github.com/Electroid/pdf2json/pull/1/commits/69ecd477d0ffe4d2e1312afc1147f6f370560ab0 fixes the PDFJS bug
https://github.com/Electroid/pdf2json/pull/1/commits/f5162660ed2fa09b02c106778f0b8ff9a68208ce & https://github.com/Electroid/pdf2json/pull/1/commits/c5725b5f58b811f352f7b948d8c8c60f38d6700e make changes so Node 18.x, 20.x, 22.x and Bun 1.x are tested in CI

Thanks for working on this package and happy to help!

Mar 04 '25 00:03 Electroid

@Electroid I'm a big fan of your work! Unfortunately I cannot take action on this PR, but the code looks good! Also, keeping support for older versions of NodeJs is always a plus. Last word goes to @modesty

Apr 03 '25 22:04 nicolabaesso