palantir / conjure-typescript-runtime

An HTTP bridge library for use in front end applications and generated conjure code
Apache License 2.0
5 stars 13 forks source link

Fix isConjureError for nullish values and add tests #146

Closed tomshen closed 2 years ago

tomshen commented 2 years ago

Before this PR

isConjureError(null) and isConjureError(undefined) would throw type errors:

TypeError: Cannot read property '__proto__' of null

After this PR

==COMMIT_MSG== Fix isConjureError handling of null/undefined values. ==COMMIT_MSG==

Possible downsides?

N/A

gluxon commented 2 years ago

Release triggered here: https://github.com/palantir/conjure-typescript-runtime/releases/tag/2.8.0