fix: Resolve missing process variable in browser environment

deepgram / deepgram-js-sdk

Official JavaScript SDK for Deepgram's automated speech recognition APIs.

https://developers.deepgram.com

MIT License

158 stars 54 forks source link

Closed noxan closed 5 months ago

noxan commented 5 months ago

Follow up to close #297

check if process is defined, else fall back to Unknown
prevent the unlikely case of process.versions being defined without a node key, fall back to Unknown

noxan commented 5 months ago

@lukeocodes added a fix with this pull request

lukeocodes commented 5 months ago

nice, this is a neater solution than a vite config

noxan commented 5 months ago

thanks for the swift response and release :)

lukeocodes commented 5 months ago

no worries, thank you for your attention to detail! made it easier to test/accept