This is a proposal for fixing #521. It works in the handful of local tests that I've done, but it probably needs more testing. I can test further if this general approach is acceptable.
Add a named export of PinoPretty - This was declared by the pre-existing types but was missing from the JS, causing runtime errors. It's easy to update the JS to match the intention of the pre-existing types.
This is a proposal for fixing #521. It works in the handful of local tests that I've done, but it probably needs more testing. I can test further if this general approach is acceptable.
See individual commits for specific changes:
declare namespace
throughout, following the "main export is a function" example from Are the Types WrongPinoPretty
- This was declared by the pre-existing types but was missing from the JS, causing runtime errors. It's easy to update the JS to match the intention of the pre-existing types.Fixes #521